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 | |
| parent | bc178622d40d87e75abc131007342429c9b03351 (diff) | |
| download | op-kernel-dev-492104c866cb1b62a11393adccb477f5cd2c7768.zip op-kernel-dev-492104c866cb1b62a11393adccb477f5cd2c7768.tar.gz | |
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')
| -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 aaee2b7..350b9b1 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, if (ret && !insert) { err = -ENOENT; goto out; + } else if (ret) { + err = -EIO; + WARN_ON(1); + goto out; } - BUG_ON(ret); /* Corruption */ leaf = path->nodes[0]; item_size = btrfs_item_size_nr(leaf, path->slots[0]); |
