btrfs: exit after state split error at btrfs_convert_extent_bit()
authorFilipe Manana <fdmanana@suse.com>
Thu, 10 Apr 2025 16:01:59 +0000 (17:01 +0100)
committerDavid Sterba <dsterba@suse.com>
Thu, 15 May 2025 12:30:51 +0000 (14:30 +0200)
commit2a72dd999610f9a8e2ddc0eaaf054cd66e45cde7
tree7a7abb47c2126ec0818438d5628e365694ef1329
parent5f9c554a6c1219dee1a3484db2e19881d35a0788
btrfs: exit after state split error at btrfs_convert_extent_bit()

If split_state() returned an error we call extent_io_tree_panic() which
will trigger a BUG() call. However if CONFIG_BUG is disabled, which is an
uncommon and exotic scenario, then we fallthrough and hit a use after free
when calling set_state_bits() since the extent state record which the
local variable 'prealloc' points to was freed by split_state().

So jump to the label 'out' after calling extent_io_tree_panic() and set
the 'prealloc' pointer to NULL since split_state() has already freed it
when it hit an error.

Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/extent-io-tree.c