]> git.dujemihanovic.xyz Git - u-boot.git/commit
lmb: do not panic in lmb_print_region_flags
authorHeinrich Schuchardt <heinrich.schuchardt@canonical.com>
Sat, 2 Nov 2024 06:32:26 +0000 (07:32 +0100)
committerTom Rini <trini@konsulko.com>
Fri, 15 Nov 2024 00:14:05 +0000 (18:14 -0600)
commit1f66c0e1f4ffb2f9f0fea8fcb7118028039e76a2
tree84dfed375fb2b9e4d8d476bc4f51a7058c43545a
parent1b99c15d73c10a7f5953e7cd69264754f5f604ba
lmb: do not panic in lmb_print_region_flags

Commit c3cf0dc64f1c ("lmb: add a check to prevent memory overrun")
addressed a possible buffer overrun using assert_noisy().

Resetting via panic() in lmb_print_region() while allowing invalid
lmb flags elsewhere is not reasonable.

Instead of panicking print a message indicating the problem.

fls() returns an int. Using a u64 for bitpos does not match.
Use int instead.

fls() takes an int as argument. Using 1ull << bitpos generates a u64.
Use 1u << bitpos instead.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Acked-by: Sughosh Ganu <sughosh.ganu@linaro.org>
lib/lmb.c