diff options
| author |   Filipe Manana <fdmanana@suse.com> | 2025-02-20 10:58:38 +0000 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2025-03-18 20:35:46 +0100 |
| commit | 26605cc9d0d556618893a5e05abde9b36efe7a3e (patch) | |
| tree | bc539f56c547914b8be0b843cbfe7557b1c9e2ed /fs/btrfs/send.c | |
| parent | btrfs: send: simplify return logic from send_link() (diff) | |
| download | wireguard-linux-26605cc9d0d556618893a5e05abde9b36efe7a3e.tar.xz wireguard-linux-26605cc9d0d556618893a5e05abde9b36efe7a3e.zip | |
btrfs: send: simplify return logic from send_unlink()
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>
Diffstat (limited to 'fs/btrfs/send.c')
| -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 bda229c7084b..cbc9ca9db062 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c @@ -856,14 +856,13 @@ static int send_unlink(struct send_ctx *sctx, struct fs_path *path) ret = begin_cmd(sctx, BTRFS_SEND_C_UNLINK); if (ret < 0) - goto out; + return ret; TLV_PUT_PATH(sctx, BTRFS_SEND_A_PATH, path); ret = send_cmd(sctx); tlv_put_failure: -out: return ret; } |