diff options
| author | Josef Bacik <jbacik@fb.com> | 2014-06-05 16:08:45 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-06-30 20:14:03 -0700 |
| commit | efb57277b9df5bcdf99aabd8b4bafe9c4cbd7576 (patch) | |
| tree | 9b54e5fd5fbc390710cd8dc4e1e11fce88e7f2e1 /fs | |
| parent | e517155065804d25328f9f2d99a8f25e87b42749 (diff) | |
Btrfs: don't check nodes for extent items
commit 8a56457f5f8fa7c2698ffae8545214c5b96a2cb5 upstream.
The backref code was looking at nodes as well as leaves when we tried to
populate extent item entries. This is not good, and although we go away with it
for the most part because we'd skip where disk_bytenr != random_memory,
sometimes random_memory would match and suddenly boom. This fixes that problem.
Thanks,
Signed-off-by: Josef Bacik <jbacik@fb.com>
Signed-off-by: Chris Mason <clm@fb.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/btrfs/backref.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c index 10db21fa0926..1e0439111d34 100644 --- a/fs/btrfs/backref.c +++ b/fs/btrfs/backref.c @@ -984,11 +984,12 @@ again: goto out; } if (ref->count && ref->parent) { - if (extent_item_pos && !ref->inode_list) { + if (extent_item_pos && !ref->inode_list && + ref->level == 0) { u32 bsz; struct extent_buffer *eb; bsz = btrfs_level_size(fs_info->extent_root, - info_level); + ref->level); eb = read_tree_block(fs_info->extent_root, ref->parent, bsz, 0); if (!eb || !extent_buffer_uptodate(eb)) { |