vhaudiquet/u-boot-krane
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: cmd: part: fix integer handling in setup_gpt_partitions()

Browse Source 
Coverity reports an INTEGER_OVERFLOW issue because ut_asserteq() compares
an unsigned int (mmc_dev_num) with ret, which can hold a negative error
code.

Addresses-Coverity-ID: CID 646439: Integer handling issues  (INTEGER_OVERFLOW)
Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de>
This commit is contained in:
Dario Binacchi
2026-07-13 17:06:37 -06:00
committed by Tom Rini
parent ab2266aff3
commit 8d50a5ce72
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/cmd/part.c
+1 -1
View File
@@ -50,7 +50,7 @@ static int setup_gpt_partitions(struct unit_test_state *uts,
if (ret == -ENODEV)
return -EAGAIN;
ut_asserteq(mmc_dev_num, ret);
ut_assert(ret >= 0 && ret == mmc_dev_num);
if (CONFIG_IS_ENABLED(RANDOM_UUID)) {
for (i = 0; i < ARRAY_SIZE(gpt_parts); i++)