diff options
| author |   Chengguang Xu <cgxu519@gmx.com> | 2018-06-27 12:16:37 +0800 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2018-08-06 13:12:40 +0200 |
| commit | dc7789ef8785ea996ecf302a89415439ccf10faa (patch) | |
| tree | b86b925e8678202c3323c174566618693f650de0 /fs/btrfs/acl.c | |
| parent | btrfs: remove unnecessary -ERANGE check in btrfs_get_acl (diff) | |
| download | linux-dev-dc7789ef8785ea996ecf302a89415439ccf10faa.tar.xz linux-dev-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>
Diffstat (limited to 'fs/btrfs/acl.c')
| -rw-r--r-- | fs/btrfs/acl.c | 2 |
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); |