summaryrefslogtreecommitdiffstats
path: root/fs/btrfs/acl.c
diff options
context:
space:
mode:
authorChengguang Xu <cgxu519@gmx.com>2018-06-27 12:16:36 +0800
committerDavid Sterba <dsterba@suse.com>2018-08-06 13:12:40 +0200
commit5ee552da503961e6df785b8495bcfc98eb30eb2a (patch)
tree65080502f0cdc2b60f229017ae570653f5cd7ff4 /fs/btrfs/acl.c
parent7e35eab958e30d15743b1f9928d15b559a6e432d (diff)
downloadlinux-5ee552da503961e6df785b8495bcfc98eb30eb2a.tar.gz
linux-5ee552da503961e6df785b8495bcfc98eb30eb2a.tar.bz2
linux-5ee552da503961e6df785b8495bcfc98eb30eb2a.zip
btrfs: remove unnecessary -ERANGE check in btrfs_get_acl
There is no chance to get into -ERANGE error condition because we first call btrfs_getxattr to get the length of the attribute, then we do a subsequent call with the size from the first call. Between the 2 calls the size shouldn't change. So remove the unnecessary -ERANGE error check. Signed-off-by: Chengguang Xu <cgxu519@gmx.com> Reviewed-by: Nikolay Borisov <nborisov@suse.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/acl.c')
-rw-r--r--fs/btrfs/acl.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/acl.c b/fs/btrfs/acl.c
index 83fdd80c51c6..a1d7211c8884 100644
--- a/fs/btrfs/acl.c
+++ b/fs/btrfs/acl.c
@@ -42,7 +42,7 @@ struct posix_acl *btrfs_get_acl(struct inode *inode, int type)
}
if (size > 0) {
acl = posix_acl_from_xattr(&init_user_ns, value, size);
- } else if (size == -ERANGE || size == -ENODATA || size == 0) {
+ } else if (size == -ENODATA || size == 0) {
acl = NULL;
} else {
acl = ERR_PTR(-EIO);