diff options
| author |   Nikolay Borisov <nborisov@suse.com> | 2022-04-29 17:17:34 +0300 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2022-05-16 17:17:32 +0200 |
| commit | cf4f03c3be4a2a08be5d3a40d25ef577be2db061 (patch) | |
| tree | e85a40427344423db7ffbb1def66bfd6d3bdc022 /fs/btrfs | |
| parent | btrfs: rename bio_ctrl::bio_flags to compress_type (diff) | |
| download | linux-dev-cf4f03c3be4a2a08be5d3a40d25ef577be2db061.tar.xz linux-dev-cf4f03c3be4a2a08be5d3a40d25ef577be2db061.zip | |
btrfs: improve error reporting in lookup_inline_extent_backref
When iterating the backrefs in an extent item if the ptr to the
'current' backref record goes beyond the extent item a warning is
generated and -ENOENT is returned. However what's more appropriate to
debug such cases would be to return EUCLEAN and also print identifying
information about the performed search as well as the current content of
the leaf containing the possibly corrupted extent item.
Reviewed-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs')
| -rw-r--r-- | fs/btrfs/extent-tree.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c index 856c6252ef9f..8ef4aa5a46d6 100644 --- a/fs/btrfs/extent-tree.c +++ b/fs/btrfs/extent-tree.c @@ -895,7 +895,13 @@ again: err = -ENOENT; while (1) { if (ptr >= end) { - WARN_ON(ptr > end); + if (ptr > end) { + err = -EUCLEAN; + btrfs_print_leaf(path->nodes[0]); + btrfs_crit(fs_info, +"overrun extent record at slot %d while looking for inline extent for root %llu owner %llu offset %llu parent %llu", + path->slots[0], root_objectid, owner, offset, parent); + } break; } iref = (struct btrfs_extent_inline_ref *)ptr; |