diff options
| author |   Jeff Mahoney <jeffm@suse.com> | 2011-10-03 23:22:43 -0400 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.cz> | 2012-03-22 01:45:32 +0100 |
| commit | 200a5c17677e1ee8b78382046f3748c9f5816281 (patch) | |
| tree | aca0964b50af798d919ca3b710cfcd9c69498170 /fs/btrfs/disk-io.c | |
| parent | btrfs: Remove set bits return from clear_extent_bit (diff) | |
| download | linux-dev-200a5c17677e1ee8b78382046f3748c9f5816281.tar.xz linux-dev-200a5c17677e1ee8b78382046f3748c9f5816281.zip | |
btrfs: find_and_setup_root error push-up
find_and_setup_root BUGs when it encounters an error from
btrfs_find_last_root, which can occur if a path can't be allocated.
This patch pushes it up to its callers where it is already handled.
Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Diffstat (limited to 'fs/btrfs/disk-io.c')
| -rw-r--r-- | fs/btrfs/disk-io.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c index d52ec1155204..155c4e35e536 100644 --- a/fs/btrfs/disk-io.c +++ b/fs/btrfs/disk-io.c @@ -1197,10 +1197,10 @@ static int __setup_root(u32 nodesize, u32 leafsize, u32 sectorsize, return 0; } -static int find_and_setup_root(struct btrfs_root *tree_root, - struct btrfs_fs_info *fs_info, - u64 objectid, - struct btrfs_root *root) +static int __must_check find_and_setup_root(struct btrfs_root *tree_root, + struct btrfs_fs_info *fs_info, + u64 objectid, + struct btrfs_root *root) { int ret; u32 blocksize; @@ -1213,7 +1213,8 @@ static int find_and_setup_root(struct btrfs_root *tree_root, &root->root_item, &root->root_key); if (ret > 0) return -ENOENT; - BUG_ON(ret); + else if (ret < 0) + return ret; generation = btrfs_root_generation(&root->root_item); blocksize = btrfs_level_size(root, btrfs_root_level(&root->root_item)); |