diff options
author | Jan Kara <jack@suse.cz> | 2025-03-12 16:18:39 +0100 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2025-03-13 18:23:29 +0100 |
commit | 6afdc60ec30b0a9390d11b7cebed79c857ce82aa (patch) | |
tree | ea5c976e7f415516597db9781c5c0a6424fd5830 /include/linux/compiler.h | |
parent | a7624ccdef4f020325893534914fad549e51ac83 (diff) | |
download | linux-6afdc60ec30b0a9390d11b7cebed79c857ce82aa.tar.gz linux-6afdc60ec30b0a9390d11b7cebed79c857ce82aa.tar.bz2 linux-6afdc60ec30b0a9390d11b7cebed79c857ce82aa.zip |
udf: Fix inode_getblk() return value
Smatch noticed that inode_getblk() can return 1 on successful mapping of
a block instead of expected 0 after commit b405c1e58b73 ("udf: refactor
udf_next_aext() to handle error"). This could confuse some of the
callers and lead to strange failures (although the one reported by
Smatch in udf_mkdir() is impossible to trigger in practice). Fix the
return value of inode_getblk().
Link: https://lore.kernel.org/all/cb514af7-bbe0-435b-934f-dd1d7a16d2cd@stanley.mountain
Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
Fixes: b405c1e58b73 ("udf: refactor udf_next_aext() to handle error")
CC: stable@vger.kernel.org
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'include/linux/compiler.h')
0 files changed, 0 insertions, 0 deletions