]> www.infradead.org Git - nvme.git/commit
btrfs: fix uninitialized return value in the ref-verify tool
authorFilipe Manana <fdmanana@suse.com>
Sun, 23 Jun 2024 11:50:26 +0000 (12:50 +0100)
committerDavid Sterba <dsterba@suse.com>
Tue, 2 Jul 2024 17:14:57 +0000 (19:14 +0200)
commit9da45c88e124f13a3c4d480b89b298e007fbb9e4
tree33f7428729cd1f1bcc89f4d7583a0f9dfa9eda54
parent724d8042cef84496ddb4492dc120291f997ae26b
btrfs: fix uninitialized return value in the ref-verify tool

In the ref-verify tool, when processing the inline references of an extent
item, we may end up returning with uninitialized return value, because:

1) The 'ret' variable is not initialized if there are no inline extent
   references ('ptr' == 'end' before the while loop starts);

2) If we find an extent owner inline reference we don't initialize 'ret'.

So fix these cases by initializing 'ret' to 0 when declaring the variable
and set it to -EINVAL if we find an extent owner inline references and
simple quotas are not enabled (as well as print an error message).

Reported-by: Mirsad Todorovac <mtodorovac69@gmail.com>
Link: https://lore.kernel.org/linux-btrfs/59b40ebe-c824-457d-8b24-0bbca69d472b@gmail.com/
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/ref-verify.c