diff options
| author | David Sterba <dsterba@suse.com> | 2020-11-16 19:53:52 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2020-11-23 21:16:21 +0100 |
| commit | 6d06b0ad94d3dd7e3503d8ad39c39c4634884611 (patch) | |
| tree | ae2d5e50a162ab6ac772fcdcfcd193051c6b2468 /fs/btrfs/tree-checker.c | |
| parent | 0697d9a610998b8bdee6b2390836cb2391d8fd1a (diff) | |
| download | lwn-6d06b0ad94d3dd7e3503d8ad39c39c4634884611.tar.gz lwn-6d06b0ad94d3dd7e3503d8ad39c39c4634884611.zip | |
btrfs: tree-checker: add missing returns after data_ref alignment checks
There are sectorsize alignment checks that are reported but then
check_extent_data_ref continues. This was not intended, wrong alignment
is not a minor problem and we should return with error.
CC: stable@vger.kernel.org # 5.4+
Fixes: 0785a9aacf9d ("btrfs: tree-checker: Add EXTENT_DATA_REF check")
Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/tree-checker.c')
| -rw-r--r-- | fs/btrfs/tree-checker.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/btrfs/tree-checker.c b/fs/btrfs/tree-checker.c index 6cefabd27209..ea2bb4cb5890 100644 --- a/fs/btrfs/tree-checker.c +++ b/fs/btrfs/tree-checker.c @@ -1424,6 +1424,7 @@ static int check_extent_data_ref(struct extent_buffer *leaf, "invalid item size, have %u expect aligned to %zu for key type %u", btrfs_item_size_nr(leaf, slot), sizeof(*dref), key->type); + return -EUCLEAN; } if (!IS_ALIGNED(key->objectid, leaf->fs_info->sectorsize)) { generic_err(leaf, slot, @@ -1452,6 +1453,7 @@ static int check_extent_data_ref(struct extent_buffer *leaf, extent_err(leaf, slot, "invalid extent data backref offset, have %llu expect aligned to %u", offset, leaf->fs_info->sectorsize); + return -EUCLEAN; } } return 0; |