btrfs: move transaction aborts to the error site in add_block_group_free_space()

Transaction aborts should be done next to the place the error happens,
which was not done in add_block_group_free_space().

Reviewed-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

diff --git a/fs/btrfs/free-space-tree.c b/fs/btrfs/free-space-tree.c
index 0755656a270eecf5dd0ba8522363b76fea354ea3..9eb9858e8e995a159cb82310db6f32a548e1aa7f 100644 (file)
--- a/fs/btrfs/free-space-tree.c
+++ b/fs/btrfs/free-space-tree.c
@@ -1456,16 +1456,17 @@ int add_block_group_free_space(struct btrfs_trans_handle *trans,
        path = btrfs_alloc_path();
        if (!path) {
                ret = -ENOMEM;
+               btrfs_abort_transaction(trans, ret);
                goto out;
        }
 
        ret = __add_block_group_free_space(trans, block_group, path);
+       if (ret)
+               btrfs_abort_transaction(trans, ret);
 
 out:
        btrfs_free_path(path);
        mutex_unlock(&block_group->free_space_lock);
-       if (ret)
-               btrfs_abort_transaction(trans, ret);
        return ret;
 }