ceph: fix ceph_set_acl()
authorYan, Zheng <zheng.z.yan@intel.com>
Tue, 11 Feb 2014 05:08:51 +0000 (13:08 +0800)
committerSage Weil <sage@inktank.com>
Mon, 17 Feb 2014 20:37:11 +0000 (12:37 -0800)
If acl is equivalent to file mode permission bits, ceph_set_acl()
needs to remove any existing acl xattr. Use __ceph_setxattr() to
handle both setting and removing acl xattr cases, it doesn't return
-ENODATA when there is no acl xattr.

Signed-off-by: Yan, Zheng <zheng.z.yan@intel.com>
fs/ceph/acl.c

index 4c2d452c4bfc0abad11cf3e3b0577d62ed3ef240..accc9f28bc106db0cb9b9e478b800dc6b93d27d4 100644 (file)
@@ -160,11 +160,7 @@ int ceph_set_acl(struct inode *inode, struct posix_acl *acl, int type)
                        goto out_dput;
        }
 
-       if (value)
-               ret = __ceph_setxattr(dentry, name, value, size, 0);
-       else
-               ret = __ceph_removexattr(dentry, name);
-
+       ret = __ceph_setxattr(dentry, name, value, size, 0);
        if (ret) {
                if (new_mode != old_mode) {
                        newattrs.ia_mode = old_mode;