[PATCH] test: cmd: part: fix integer handling in setup_gpt_partitions()
Dario Binacchi
dario.binacchi at amarulasolutions.com
Fri Jun 26 20:33:49 CEST 2026
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 at amarulasolutions.com>
---
test/cmd/part.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/cmd/part.c b/test/cmd/part.c
index e0149011476c..b01bc286723b 100644
--- a/test/cmd/part.c
+++ b/test/cmd/part.c
@@ -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++)
--
2.43.0
base-commit: 6902fb4c17faa375003124c451c2550deab5463d
branch: test_cmd_part
More information about the U-Boot
mailing list