diff options
| author | Chengguang Xu <cgxu519@gmx.com> | 2018-08-17 22:05:31 +0800 |
|---|---|---|
| committer | Ilya Dryomov <idryomov@gmail.com> | 2020-01-27 16:53:39 +0100 |
| commit | d80865bff5201cc56bc247989ebbab6169b3a101 (patch) | |
| tree | 117a13fd4bf1661f2562a2e34ab32399db9ffecb /fs/ceph/acl.c | |
| parent | 8f5ac172abb79171eac9ecb7bedc071b56630097 (diff) | |
| download | linux-stable-d80865bff5201cc56bc247989ebbab6169b3a101.tar.gz linux-stable-d80865bff5201cc56bc247989ebbab6169b3a101.tar.bz2 linux-stable-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) |