diff options
author | Josef Bacik <jbacik@fusionio.com> | 2013-03-08 15:41:02 -0500 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2013-03-14 14:57:29 -0400 |
commit | 492104c866cb1b62a11393adccb477f5cd2c7768 (patch) | |
tree | 1aacf5abaf22d0a591b4cb39e494c95694fb589c /fs/btrfs/extent-tree.c | |
parent | bc178622d40d87e75abc131007342429c9b03351 (diff) |
Btrfs: return EIO if we have extent tree corruption
The callers of lookup_inline_extent_info all handle getting an error back
properly, so return an error if we have corruption instead of being a jerk and
panicing. Still WARN_ON() since this is kind of crucial and I've been seeing it
a bit too much recently for my taste, I think we're doing something wrong
somewhere. Thanks,
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'fs/btrfs/extent-tree.c')
-rw-r--r-- | fs/btrfs/extent-tree.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c index aaee2b7fee78..350b9b18140c 100644 --- a/fs/btrfs/extent-tree.c +++ b/fs/btrfs/extent-tree.c | |||
@@ -1467,8 +1467,11 @@ int lookup_inline_extent_backref(struct btrfs_trans_handle *trans, | |||
1467 | if (ret && !insert) { | 1467 | if (ret && !insert) { |
1468 | err = -ENOENT; | 1468 | err = -ENOENT; |
1469 | goto out; | 1469 | goto out; |
1470 | } else if (ret) { | ||
1471 | err = -EIO; | ||
1472 | WARN_ON(1); | ||
1473 | goto out; | ||
1470 | } | 1474 | } |
1471 | BUG_ON(ret); /* Corruption */ | ||
1472 | 1475 | ||
1473 | leaf = path->nodes[0]; | 1476 | leaf = path->nodes[0]; |
1474 | item_size = btrfs_item_size_nr(leaf, path->slots[0]); | 1477 | item_size = btrfs_item_size_nr(leaf, path->slots[0]); |