diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2008-07-16 09:51:03 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2008-07-26 20:53:19 -0400 |
| commit | 8bb79224b87aab92071e94d46e70bd160d89bf34 (patch) | |
| tree | dab47cbb434876e5916f18b738ef8bd3fe1e1ce1 /fs/open.c | |
| parent | db2e747b14991a4c6a5c98b0e5f552a193237c03 (diff) | |
| download | lwn-8bb79224b87aab92071e94d46e70bd160d89bf34.tar.gz lwn-8bb79224b87aab92071e94d46e70bd160d89bf34.zip | |
[PATCH] permission checks for chdir need special treatment only on the last step
... so we ought to pass MAY_CHDIR to vfs_permission() instead of having
it triggered on every step of preceding pathname resolution. LOOKUP_CHDIR
is killed by that.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/open.c')
| -rw-r--r-- | fs/open.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/open.c b/fs/open.c index b2e4c93aed03..8e02d42bfe44 100644 --- a/fs/open.c +++ b/fs/open.c @@ -501,12 +501,11 @@ asmlinkage long sys_chdir(const char __user * filename) struct nameidata nd; int error; - error = __user_walk(filename, - LOOKUP_FOLLOW|LOOKUP_DIRECTORY|LOOKUP_CHDIR, &nd); + error = __user_walk(filename, LOOKUP_FOLLOW|LOOKUP_DIRECTORY, &nd); if (error) goto out; - error = vfs_permission(&nd, MAY_EXEC); + error = vfs_permission(&nd, MAY_EXEC | MAY_CHDIR); if (error) goto dput_and_out; |