diff options
| author | Filipe Manana <fdmanana@suse.com> | 2025-05-16 19:13:53 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2025-07-21 15:31:06 +0200 |
| commit | 5ff6050fcd3c5b8ca16beb058af81186ac6f67fb (patch) | |
| tree | f2f28c25addcf60d7c1621549749df20b442c9b4 /fs/btrfs/reflink.c | |
| parent | 5cf0e668ea8a64b45918aee38f5602b9dbbc3e7e (diff) | |
btrfs: remove pointless 'out' label from clone_finish_inode_update()
The label is only used once and we can instead return directly where it's
used, besides the fact that all we do under the label is to return the
value of 'ret'. So get rid of the label and return directly.
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.cz>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/reflink.c')
| -rw-r--r-- | fs/btrfs/reflink.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/btrfs/reflink.c b/fs/btrfs/reflink.c index 62161beca559..5eacd3584a8d 100644 --- a/fs/btrfs/reflink.c +++ b/fs/btrfs/reflink.c @@ -46,11 +46,9 @@ static int clone_finish_inode_update(struct btrfs_trans_handle *trans, if (ret) { btrfs_abort_transaction(trans, ret); btrfs_end_transaction(trans); - goto out; + return ret; } - ret = btrfs_end_transaction(trans); -out: - return ret; + return btrfs_end_transaction(trans); } static int copy_inline_to_page(struct btrfs_inode *inode, |