diff options
author | Filipe David Borba Manana <fdmanana@gmail.com> | 2013-07-08 14:21:24 +0100 |
---|---|---|
committer | David Sterba <dsterba@suse.cz> | 2013-08-09 14:32:33 +0200 |
commit | 7eb13593e39ae353d847047433fe257b3aab05a0 (patch) | |
tree | c2b54bfb9938aa4e8147ce38630ed7852f6af777 | |
parent | 2e9adfb90975852ce46692f0d433e3a9fcc78749 (diff) |
Btrfs-progs: don't ignore errors in extent-tree.c
Several function return values were being completely
ignored.
Signed-off-by: Filipe David Borba Manana <fdmanana@gmail.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
-rw-r--r-- | extent-tree.c | 20 |
1 files changed, 13 insertions, 7 deletions
diff --git a/extent-tree.c b/extent-tree.c index a89f525c..3f54706a 100644 --- a/extent-tree.c +++ b/extent-tree.c @@ -2108,6 +2108,7 @@ static int finish_current_insert(struct btrfs_trans_handle *trans, extent_op->flags, &extent_op->key, extent_op->level, &key); + BUG_ON(ret); } else { BUG_ON(1); } @@ -2755,7 +2756,7 @@ static int alloc_reserved_tree_block(struct btrfs_trans_handle *trans, ret = update_block_group(trans, root, ins->objectid, root->leafsize, 1, 0); - return 0; + return ret; } static int alloc_tree_block(struct btrfs_trans_handle *trans, @@ -3274,12 +3275,14 @@ btrfs_add_block_group(struct btrfs_fs_info *fs_info, u64 bytes_used, u64 type, BUG_ON(ret); bit = block_group_state_bits(type); - set_extent_bits(block_group_cache, chunk_offset, - chunk_offset + size - 1, - bit | EXTENT_LOCKED, GFP_NOFS); + ret = set_extent_bits(block_group_cache, chunk_offset, + chunk_offset + size - 1, + bit | EXTENT_LOCKED, GFP_NOFS); + BUG_ON(ret); - set_state_private(block_group_cache, chunk_offset, - (unsigned long)cache); + ret = set_state_private(block_group_cache, chunk_offset, + (unsigned long)cache); + BUG_ON(ret); set_avail_alloc_bits(fs_info, type); return cache; @@ -3301,8 +3304,11 @@ int btrfs_make_block_group(struct btrfs_trans_handle *trans, sizeof(cache->item)); BUG_ON(ret); - finish_current_insert(trans, extent_root); + ret = finish_current_insert(trans, extent_root); + BUG_ON(ret); ret = del_pending_extents(trans, extent_root); + BUG_ON(ret); + return 0; } |