diff options
| author |   Filipe Manana <fdmanana@suse.com> | 2025-02-20 10:57:57 +0000 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2025-03-18 20:35:46 +0100 |
| commit | 711584496f959d4347734d6e8ab920f5b11cfc14 (patch) | |
| tree | 821a3634aec990d03d961192a3b1f5f585e23681 | |
| parent | btrfs: send: simplify return logic from send_rename() (diff) | |
| download | wireguard-linux-711584496f959d4347734d6e8ab920f5b11cfc14.tar.xz wireguard-linux-711584496f959d4347734d6e8ab920f5b11cfc14.zip | |
btrfs: send: simplify return logic from send_link()
There is no need to have an 'out' label and jump into it since there are
no resource cleanups to perform (release locks, free memory, etc), so
make this simpler by removing the label and goto and instead return
directly.
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/send.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c index d5c151651d07..bda229c7084b 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c @@ -833,7 +833,7 @@ static int send_link(struct send_ctx *sctx, ret = begin_cmd(sctx, BTRFS_SEND_C_LINK); if (ret < 0) - goto out; + return ret; TLV_PUT_PATH(sctx, BTRFS_SEND_A_PATH, path); TLV_PUT_PATH(sctx, BTRFS_SEND_A_PATH_LINK, lnk); @@ -841,7 +841,6 @@ static int send_link(struct send_ctx *sctx, ret = send_cmd(sctx); tlv_put_failure: -out: return ret; } |