mirror of
https://github.com/torvalds/linux.git
synced 2025-11-01 00:58:39 +02:00
btrfs: send: simplify return logic from send_remove_xattr()
There's no need for the 'out' label as there are no resources to cleanup in case of an error and we can directly return if begin_cmd() fails. Signed-off-by: Filipe Manana <fdmanana@suse.com> Reviewed-by: David Sterba <dsterba@suse.com> Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
parent
ec666c84de
commit
9435159f28
1 changed files with 2 additions and 3 deletions
|
|
@ -4886,11 +4886,11 @@ static int send_remove_xattr(struct send_ctx *sctx,
|
||||||
struct fs_path *path,
|
struct fs_path *path,
|
||||||
const char *name, int name_len)
|
const char *name, int name_len)
|
||||||
{
|
{
|
||||||
int ret = 0;
|
int ret;
|
||||||
|
|
||||||
ret = begin_cmd(sctx, BTRFS_SEND_C_REMOVE_XATTR);
|
ret = begin_cmd(sctx, BTRFS_SEND_C_REMOVE_XATTR);
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
goto out;
|
return ret;
|
||||||
|
|
||||||
TLV_PUT_PATH(sctx, BTRFS_SEND_A_PATH, path);
|
TLV_PUT_PATH(sctx, BTRFS_SEND_A_PATH, path);
|
||||||
TLV_PUT_STRING(sctx, BTRFS_SEND_A_XATTR_NAME, name, name_len);
|
TLV_PUT_STRING(sctx, BTRFS_SEND_A_XATTR_NAME, name, name_len);
|
||||||
|
|
@ -4898,7 +4898,6 @@ static int send_remove_xattr(struct send_ctx *sctx,
|
||||||
ret = send_cmd(sctx);
|
ret = send_cmd(sctx);
|
||||||
|
|
||||||
tlv_put_failure:
|
tlv_put_failure:
|
||||||
out:
|
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue