diff options
author | 2018-08-17 22:05:31 +0800 | |
---|---|---|
committer | 2020-01-27 16:53:39 +0100 | |
commit | d80865bff5201cc56bc247989ebbab6169b3a101 (patch) | |
tree | 117a13fd4bf1661f2562a2e34ab32399db9ffecb /fs/ceph/acl.c | |
parent | ceph: delete redundant douts in con_get/put() (diff) | |
download | linux-d80865bff5201cc56bc247989ebbab6169b3a101.tar.gz linux-d80865bff5201cc56bc247989ebbab6169b3a101.tar.bz2 linux-d80865bff5201cc56bc247989ebbab6169b3a101.zip |
ceph: remove unnecessary assignment in ceph_pre_init_acls()
ceph_pagelist_encode_string() will not fail in reserved case,
also, we do not check err code here, so remove unnecessary
assignment.
Signed-off-by: Chengguang Xu <cgxu519@gmx.com>
Reviewed-by: "Yan, Zheng" <zyan@redhat.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'fs/ceph/acl.c')
-rw-r--r-- | fs/ceph/acl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/ceph/acl.c b/fs/ceph/acl.c index aa55f412a6e3..26be6520d3fb 100644 --- a/fs/ceph/acl.c +++ b/fs/ceph/acl.c @@ -222,8 +222,8 @@ int ceph_pre_init_acls(struct inode *dir, umode_t *mode, err = ceph_pagelist_reserve(pagelist, len + val_size2 + 8); if (err) goto out_err; - err = ceph_pagelist_encode_string(pagelist, - XATTR_NAME_POSIX_ACL_DEFAULT, len); + ceph_pagelist_encode_string(pagelist, + XATTR_NAME_POSIX_ACL_DEFAULT, len); err = posix_acl_to_xattr(&init_user_ns, default_acl, tmp_buf, val_size2); if (err < 0) |