summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChengguang Xu <cgxu519@gmx.com>2018-06-27 12:16:37 +0800
committerDavid Sterba <dsterba@suse.com>2018-08-06 13:12:40 +0200
commitdc7789ef8785ea996ecf302a89415439ccf10faa (patch)
treeb86b925e8678202c3323c174566618693f650de0
parent5ee552da503961e6df785b8495bcfc98eb30eb2a (diff)
downloadlinux-dc7789ef8785ea996ecf302a89415439ccf10faa.tar.gz
linux-dc7789ef8785ea996ecf302a89415439ccf10faa.tar.bz2
linux-dc7789ef8785ea996ecf302a89415439ccf10faa.zip
btrfs: avoid error code override in btrfs_get_acl
It's not good to override the error code when failing from btrfs_getxattr() in btrfs_get_acl() because it hides the real reason of the failure. 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>
-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 a1d7211c8884..7d673ec9e54a 100644
--- a/fs/btrfs/acl.c
+++ b/fs/btrfs/acl.c
@@ -45,7 +45,7 @@ struct posix_acl *btrfs_get_acl(struct inode *inode, int type)
} else if (size == -ENODATA || size == 0) {
acl = NULL;
} else {
- acl = ERR_PTR(-EIO);
+ acl = ERR_PTR(size);
}
kfree(value);