diff options
| author |   Filipe Manana <fdmanana@suse.com> | 2025-02-20 10:59:13 +0000 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2025-03-18 20:35:46 +0100 |
| commit | d7d56ccf100adb140c5655c624fedf7545d569e0 (patch) | |
| tree | 8237f44ca4f156541e1b75defd89588427a89d3d | |
| parent | btrfs: send: simplify return logic from send_unlink() (diff) | |
| download | wireguard-linux-d7d56ccf100adb140c5655c624fedf7545d569e0.tar.xz wireguard-linux-d7d56ccf100adb140c5655c624fedf7545d569e0.zip | |
btrfs: send: simplify return logic from send_rmdir()
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 cbc9ca9db062..0c496270e10f 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c @@ -878,14 +878,13 @@ static int send_rmdir(struct send_ctx *sctx, struct fs_path *path) ret = begin_cmd(sctx, BTRFS_SEND_C_RMDIR); 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; } |