diff options
author | Zhang Zhen <zhenzhang.zhang@huawei.com> | 2014-04-10 01:29:17 -0400 |
---|---|---|
committer | Yan, Zheng <zheng.z.yan@intel.com> | 2014-06-05 21:29:50 -0400 |
commit | 979d4c1895474aafa473f441846dd3b6f7276202 (patch) | |
tree | cead35c5a059c604263d17df5b8728636ec08d71 /fs | |
parent | e84be11c5360df33f0ba6c39185ccba564ec7177 (diff) |
ceph: remove useless ACL check
posix_acl_xattr_set() already does the check, and it's the only
way to feed in an ACL from userspace.
So the check here is useless, remove it.
Signed-off-by: zhang zhen <zhenzhang.zhang@huawei.com>
Reviewed-by: Yan, Zheng <zheng.z.yan@intel.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/ceph/acl.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/fs/ceph/acl.c b/fs/ceph/acl.c index 21887d63dad5..469f2e8657e8 100644 --- a/fs/ceph/acl.c +++ b/fs/ceph/acl.c | |||
@@ -104,12 +104,6 @@ int ceph_set_acl(struct inode *inode, struct posix_acl *acl, int type) | |||
104 | umode_t new_mode = inode->i_mode, old_mode = inode->i_mode; | 104 | umode_t new_mode = inode->i_mode, old_mode = inode->i_mode; |
105 | struct dentry *dentry; | 105 | struct dentry *dentry; |
106 | 106 | ||
107 | if (acl) { | ||
108 | ret = posix_acl_valid(acl); | ||
109 | if (ret < 0) | ||
110 | goto out; | ||
111 | } | ||
112 | |||
113 | switch (type) { | 107 | switch (type) { |
114 | case ACL_TYPE_ACCESS: | 108 | case ACL_TYPE_ACCESS: |
115 | name = POSIX_ACL_XATTR_ACCESS; | 109 | name = POSIX_ACL_XATTR_ACCESS; |