diff options
author | Tsutomu Itoh <t-itoh@jp.fujitsu.com> | 2011-05-19 00:37:44 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2011-05-23 13:24:39 -0400 |
commit | 65a246c5ffe3b487a001de025816326939e63362 (patch) | |
tree | 52da5b08a584ac3178f8bd35a5e7333f5a3908e1 /fs/btrfs/tree-log.c | |
parent | b0b802d7e34b0b4a78f911c3a8aad88aa91fd7ab (diff) |
Btrfs: return error code to caller when btrfs_del_item fails
The error code is returned instead of calling BUG_ON when
btrfs_del_item returns the error.
Signed-off-by: Tsutomu Itoh <t-itoh@jp.fujitsu.com>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/tree-log.c')
-rw-r--r-- | fs/btrfs/tree-log.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index f997ec0c1ba4..cf2baeb70462 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c | |||
@@ -1050,7 +1050,8 @@ static noinline int fixup_inode_link_counts(struct btrfs_trans_handle *trans, | |||
1050 | break; | 1050 | break; |
1051 | 1051 | ||
1052 | ret = btrfs_del_item(trans, root, path); | 1052 | ret = btrfs_del_item(trans, root, path); |
1053 | BUG_ON(ret); | 1053 | if (ret) |
1054 | goto out; | ||
1054 | 1055 | ||
1055 | btrfs_release_path(root, path); | 1056 | btrfs_release_path(root, path); |
1056 | inode = read_one_inode(root, key.offset); | 1057 | inode = read_one_inode(root, key.offset); |
@@ -1068,8 +1069,10 @@ static noinline int fixup_inode_link_counts(struct btrfs_trans_handle *trans, | |||
1068 | */ | 1069 | */ |
1069 | key.offset = (u64)-1; | 1070 | key.offset = (u64)-1; |
1070 | } | 1071 | } |
1072 | ret = 0; | ||
1073 | out: | ||
1071 | btrfs_release_path(root, path); | 1074 | btrfs_release_path(root, path); |
1072 | return 0; | 1075 | return ret; |
1073 | } | 1076 | } |
1074 | 1077 | ||
1075 | 1078 | ||
@@ -2587,7 +2590,8 @@ static int drop_objectid_items(struct btrfs_trans_handle *trans, | |||
2587 | break; | 2590 | break; |
2588 | 2591 | ||
2589 | ret = btrfs_del_item(trans, log, path); | 2592 | ret = btrfs_del_item(trans, log, path); |
2590 | BUG_ON(ret); | 2593 | if (ret) |
2594 | break; | ||
2591 | btrfs_release_path(log, path); | 2595 | btrfs_release_path(log, path); |
2592 | } | 2596 | } |
2593 | btrfs_release_path(log, path); | 2597 | btrfs_release_path(log, path); |