diff options
author | Liu Bo <bo.li.liu@oracle.com> | 2013-03-04 16:25:38 (GMT) |
---|---|---|
committer | Josef Bacik <jbacik@fusionio.com> | 2013-03-04 21:33:23 (GMT) |
commit | e1a1267054ea9dcafd01636e52d441f809efad5e (patch) | |
tree | 81c66abefae5b7c65d30a9535c73425da3375fde /fs/btrfs/relocation.c | |
parent | aca1bba6f9bc550a33312b28e98b3de5ddb3bb15 (diff) | |
download | linux-fsl-qoriq-e1a1267054ea9dcafd01636e52d441f809efad5e.tar.xz |
Btrfs: free all recorded tree blocks on error
We've missed the 'free blocks' part on ENOMEM error.
Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Diffstat (limited to 'fs/btrfs/relocation.c')
-rw-r--r-- | fs/btrfs/relocation.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/fs/btrfs/relocation.c b/fs/btrfs/relocation.c index 0f001c1..7d1654c 100644 --- a/fs/btrfs/relocation.c +++ b/fs/btrfs/relocation.c @@ -2848,8 +2848,10 @@ int relocate_tree_blocks(struct btrfs_trans_handle *trans, int err = 0; path = btrfs_alloc_path(); - if (!path) - return -ENOMEM; + if (!path) { + err = -ENOMEM; + goto out_path; + } rb_node = rb_first(blocks); while (rb_node) { @@ -2888,10 +2890,11 @@ int relocate_tree_blocks(struct btrfs_trans_handle *trans, rb_node = rb_next(rb_node); } out: - free_block_list(blocks); err = finish_pending_nodes(trans, rc, path, err); btrfs_free_path(path); +out_path: + free_block_list(blocks); return err; } |