summary refs log tree commit diff
path: root/mm/secretmem.c
diff options
context:
space:
mode:
authorFilipe Manana <fdmanana@suse.com>2023-06-30 16:03:44 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2023-08-11 12:08:24 +0200
commit23e72231f8281505883514b23709076e234d4f27 (patch)
tree6ba57cdc278b54a173fc610ce5c696f243377ef4 /mm/secretmem.c
parent56c0d76a97222f4a91912d43814c7129010f4725 (diff)
downloadlinux-23e72231f8281505883514b23709076e234d4f27.tar.gz
btrfs: remove BUG_ON()'s in add_new_free_space()
commit d8ccbd21918fd7fa6ce3226cffc22c444228e8ad upstream.

At add_new_free_space() we have these BUG_ON()'s that are there to deal
with any failure to add free space to the in memory free space cache.
Such failures are mostly -ENOMEM that should be very rare. However there's
no need to have these BUG_ON()'s, we can just return any error to the
caller and all callers and their upper call chain are already dealing with
errors.

So just make add_new_free_space() return any errors, while removing the
BUG_ON()'s, and returning the total amount of added free space to an
optional u64 pointer argument.

Reported-by: syzbot+3ba856e07b7127889d8c@syzkaller.appspotmail.com
Link: https://lore.kernel.org/linux-btrfs/000000000000e9cb8305ff4e8327@google.com/
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'mm/secretmem.c')
0 files changed, 0 insertions, 0 deletions