diff options
author | Christoph Hellwig <hch@lst.de> | 2011-07-23 11:36:38 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2011-07-25 14:25:38 -0400 |
commit | ebbb0ef2871bf3f529987313a9146cc42419a8a8 (patch) | |
tree | b833325b05b6e25c976be34f3be69088dbd6d7c1 /fs/9p | |
parent | e77819e57f0817c6dc7cadd061acd70c604cbce2 (diff) |
9p: do no return 0 from ->check_acl without actually checking
If we do not want to use ACLs we at least need to perform normal Unix
permission checks. From the comment I'm not quite sure that's what
is intended, but if 0p wants to do permission checks entirely on the
server it needs to do so in ->permission, not in ->check_acl.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/9p')
-rw-r--r-- | fs/9p/acl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/9p/acl.c b/fs/9p/acl.c index 8be87857605c..df4d7a171d7f 100644 --- a/fs/9p/acl.c +++ b/fs/9p/acl.c | |||
@@ -108,7 +108,7 @@ int v9fs_check_acl(struct inode *inode, int mask) | |||
108 | * On access = client and acl = on mode get the acl | 108 | * On access = client and acl = on mode get the acl |
109 | * values from the server | 109 | * values from the server |
110 | */ | 110 | */ |
111 | return 0; | 111 | return -EAGAIN; |
112 | } | 112 | } |
113 | acl = v9fs_get_cached_acl(inode, ACL_TYPE_ACCESS); | 113 | acl = v9fs_get_cached_acl(inode, ACL_TYPE_ACCESS); |
114 | 114 | ||