9fans - fans of the OS Plan 9 from Bell Labs
 help / color / mirror / Atom feed
From: ori@eigenstate.org
To: eekee57@fastmail.fm, 9fans@9fans.net
Subject: Re: [9fans] `test -x` returns wrong results for directories
Date: Sat, 6 Jun 2020 14:13:17 -0700	[thread overview]
Message-ID: <58181689FAB504813F509082A0F70248@eigenstate.org> (raw)
In-Reply-To: <33fdd954-db9a-4273-a0ac-907f07b42499@www.fastmail.com>

> it's open() which is failing. i suppose it should.
> 
> if the open fails, maybe access should stat the file, and if it's a
> directory, try dirread(2). or maybe just opening it for reading will
> work. i don't know, i'm new to this bit of plan 9 & i haven't slept.

This is a bit subtle, though -- if you want
to check if *you* can traverse the directory,
then simply checking for an execute bit isn't
enough -- you need to check that you belong to
a group that has the execute bit.

On a related note, execute permission seems
to behave strangely in general. You can create
a directory and list it:


Even without execute permission it's listable.
Makes sense.

	cpu% mkdir -p d/a/b
	cpu% touch d/x
	cpu% chmod -x d
	cpu% ls -ld d
	d-rw-r--r-- M 81 ori ori 0 Jun  5 07:53 d
	cpu% ls d
	d/a
	d/x

As expected, it can't be traversed:

	cpu% ls d/x
	ls: d/x: 'd/x' does not exist

But, you can cd into it:

	cpu% cd d

And list '.':

	cpu% ls
	a
	x
	cpu% ls .
	a
	x

Can't traverse it to list subdirectories,
though:

	ls a
	ls: a: 'a' access permission denied
	cd a
	Can't cd a: 'a' access permission denied
	cpu% cd ..
	Can't cd ..: '..' access permission denied

So, cd'ing into a directory withut +x leads
to an inescapabler trap.

Perhaps 'cd' should prevent entering that
directory.


  parent reply	other threads:[~2020-06-06 21:13 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-05 12:45 Ethan Gardener
2020-06-05 19:22 ` [9fans] " Richard Miller
2020-06-06  6:36   ` Ethan Gardener
2020-06-06 12:11     ` Alexander Kapshuk
2020-06-06 13:25     ` Charles Forsyth
2020-06-07 13:56       ` Ethan Gardener
2020-06-06 21:13     ` ori [this message]
2020-06-07 14:48       ` Ethan Gardener
2020-06-07 15:41         ` Richard Miller
2020-06-08  2:04           ` ori
2020-06-08  2:13             ` Charles Forsyth
2020-06-08  2:20               ` Charles Forsyth
2020-06-09 10:50               ` Ethan Gardener

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=58181689FAB504813F509082A0F70248@eigenstate.org \
    --to=ori@eigenstate.org \
    --cc=9fans@9fans.net \
    --cc=eekee57@fastmail.fm \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).