Browse Source

Btrfs: add tree block level sanity check

With a users corrupted fs I was getting weird behavior and panics and it turns
out it was because one of his tree blocks had a bogus header level.  So add this
to the sanity checks in the endio handler for tree blocks.  Thanks,

Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Josef Bacik 12 years ago
parent
commit
1c24c3ce6a
1 changed files with 6 additions and 0 deletions
  1. 6 0
      fs/btrfs/disk-io.c

+ 6 - 0
fs/btrfs/disk-io.c

@@ -613,6 +613,12 @@ static int btree_readpage_end_io_hook(struct page *page, u64 start, u64 end,
 		goto err;
 		goto err;
 	}
 	}
 	found_level = btrfs_header_level(eb);
 	found_level = btrfs_header_level(eb);
+	if (found_level >= BTRFS_MAX_LEVEL) {
+		btrfs_info(root->fs_info, "bad tree block level %d\n",
+			   (int)btrfs_header_level(eb));
+		ret = -EIO;
+		goto err;
+	}
 
 
 	btrfs_set_buffer_lockdep_class(btrfs_header_owner(eb),
 	btrfs_set_buffer_lockdep_class(btrfs_header_owner(eb),
 				       eb, found_level);
 				       eb, found_level);