btrfs: remove useless ACL check
authorZhangZhen <zhenzhang.zhang@huawei.com>
Tue, 8 Apr 2014 11:30:19 +0000 (19:30 +0800)
committerChris Mason <clm@fb.com>
Tue, 10 Jun 2014 00:20:42 +0000 (17:20 -0700)
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>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/acl.c

index ff9b399..9a0124a 100644 (file)
@@ -79,13 +79,6 @@ static int __btrfs_set_acl(struct btrfs_trans_handle *trans,
        const char *name;
        char *value = NULL;
 
-       if (acl) {
-               ret = posix_acl_valid(acl);
-               if (ret < 0)
-                       return ret;
-               ret = 0;
-       }
-
        switch (type) {
        case ACL_TYPE_ACCESS:
                name = POSIX_ACL_XATTR_ACCESS;