diff options
author | Christoph Hellwig <hch@lst.de> | 2011-07-23 11:37:31 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2011-07-25 14:30:23 -0400 |
commit | 4e34e719e457f2e031297175410fc0bd4016a085 (patch) | |
tree | ab969a371e0d2efc6bfbf503ca6cdfce3af3bf6c /fs/gfs2/acl.c | |
parent | edde854e8bb34a7f32fa993d721f1da0faf64165 (diff) |
fs: take the ACL checks to common code
Replace the ->check_acl method with a ->get_acl method that simply reads an
ACL from disk after having a cache miss. This means we can replace the ACL
checking boilerplate code with a single implementation in namei.c.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/gfs2/acl.c')
-rw-r--r-- | fs/gfs2/acl.c | 25 |
1 files changed, 2 insertions, 23 deletions
diff --git a/fs/gfs2/acl.c b/fs/gfs2/acl.c index a2dd63c0c11a..884c9af0542f 100644 --- a/fs/gfs2/acl.c +++ b/fs/gfs2/acl.c | |||
@@ -67,30 +67,9 @@ static struct posix_acl *gfs2_acl_get(struct gfs2_inode *ip, int type) | |||
67 | return acl; | 67 | return acl; |
68 | } | 68 | } |
69 | 69 | ||
70 | /** | 70 | struct posix_acl *gfs2_get_acl(struct inode *inode, int type) |
71 | * gfs2_check_acl - Check an ACL to see if we're allowed to do something | ||
72 | * @inode: the file we want to do something to | ||
73 | * @mask: what we want to do | ||
74 | * | ||
75 | * Returns: errno | ||
76 | */ | ||
77 | |||
78 | int gfs2_check_acl(struct inode *inode, int mask) | ||
79 | { | 71 | { |
80 | struct posix_acl *acl; | 72 | return gfs2_acl_get(GFS2_I(inode), type); |
81 | int error; | ||
82 | |||
83 | acl = gfs2_acl_get(GFS2_I(inode), ACL_TYPE_ACCESS); | ||
84 | if (IS_ERR(acl)) | ||
85 | return PTR_ERR(acl); | ||
86 | |||
87 | if (acl) { | ||
88 | error = posix_acl_permission(inode, acl, mask); | ||
89 | posix_acl_release(acl); | ||
90 | return error; | ||
91 | } | ||
92 | |||
93 | return -EAGAIN; | ||
94 | } | 73 | } |
95 | 74 | ||
96 | static int gfs2_set_mode(struct inode *inode, mode_t mode) | 75 | static int gfs2_set_mode(struct inode *inode, mode_t mode) |