btrfs: raid56: properly handle the error when unable to find the missing stripe
authorQu Wenruo <wqu@suse.com>
Mon, 10 Oct 2022 10:36:06 +0000 (18:36 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 26 Nov 2022 08:24:31 +0000 (09:24 +0100)
commit49ca2227c47baa8056ba3d64a2535e2d2ff5ebff
tree9131833f6cc6157544212af2a770a0e14e508cc4
parent0f7bd3a2dfe10b52ecd80f389f17d677f1c96863
btrfs: raid56: properly handle the error when unable to find the missing stripe

[ Upstream commit f15fb2cd979a07fbfc666e2f04b8b30ec9233b2a ]

In raid56_alloc_missing_rbio(), if we can not determine where the
missing device is inside the full stripe, we just BUG_ON().

This is not necessary especially the only caller inside scrub.c is
already properly checking the return value, and will treat it as a
memory allocation failure.

Fix the error handling by:

- Add an extra warning for the reason
  Although personally speaking it may be better to be an ASSERT().

- Properly free the allocated rbio

Signed-off-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
fs/btrfs/raid56.c