]> www.infradead.org Git - users/jedix/linux-maple.git/commit
btrfs: tree-checker: reject inline extent items with 0 ref count
authorQu Wenruo <wqu@suse.com>
Wed, 4 Dec 2024 03:00:46 +0000 (13:30 +1030)
committerDavid Sterba <dsterba@suse.com>
Tue, 17 Dec 2024 18:54:32 +0000 (19:54 +0100)
commitdfb92681a19e1d5172420baa242806414b3eff6f
treee41a70b4d38cdf17fd4831a52d536dfa1d934cac
parentbe691b5e593f2cc8cef67bbc59c1fb91b74a86a9
btrfs: tree-checker: reject inline extent items with 0 ref count

[BUG]
There is a bug report in the mailing list where btrfs_run_delayed_refs()
failed to drop the ref count for logical 25870311358464 num_bytes
2113536.

The involved leaf dump looks like this:

  item 166 key (25870311358464 168 2113536) itemoff 10091 itemsize 50
    extent refs 1 gen 84178 flags 1
    ref#0: shared data backref parent 32399126528000 count 0 <<<
    ref#1: shared data backref parent 31808973717504 count 1

Notice the count number is 0.

[CAUSE]
There is no concrete evidence yet, but considering 0 -> 1 is also a
single bit flipped, it's possible that hardware memory bitflip is
involved, causing the on-disk extent tree to be corrupted.

[FIX]
To prevent us reading such corrupted extent item, or writing such
damaged extent item back to disk, enhance the handling of
BTRFS_EXTENT_DATA_REF_KEY and BTRFS_SHARED_DATA_REF_KEY keys for both
inlined and key items, to detect such 0 ref count and reject them.

CC: stable@vger.kernel.org # 5.4+
Link: https://lore.kernel.org/linux-btrfs/7c69dd49-c346-4806-86e7-e6f863a66f48@app.fastmail.com/
Reported-by: Frankie Fisher <frankie@terrorise.me.uk>
Reviewed-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: Qu Wenruo <wqu@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/tree-checker.c