The error is only returned when kmalloc() fails. Therefore, to return the error to the caller directly here, I changed the code.
Signed-off-by: Tsutomu Itoh <[email protected]> --- fs/btrfs/acl.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/fs/btrfs/acl.c b/fs/btrfs/acl.c index 761e2cd..74ee421 100644 --- a/fs/btrfs/acl.c +++ b/fs/btrfs/acl.c @@ -136,10 +136,8 @@ static int btrfs_set_acl(struct btrfs_trans_handle *trans, if (acl) { size = posix_acl_xattr_size(acl->a_count); value = kmalloc(size, GFP_NOFS); - if (!value) { - ret = -ENOMEM; - goto out; - } + if (!value) + return -ENOMEM; ret = posix_acl_to_xattr(acl, value, size); if (ret < 0) -- 1.7.12.2 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
