[PATCH] test: cmd: part: fix integer handling in setup_gpt_partitions()
Quentin Schulz
quentin.schulz at cherry.de
Wed Jul 1 14:52:05 CEST 2026
Hi Tom,
On 6/26/26 11:58 PM, Tom Rini wrote:
> On Fri, Jun 26, 2026 at 08:40:36PM +0200, Quentin Schulz wrote:
>> Hi Dario,
>>
>> On 6/26/26 8:33 PM, Dario Binacchi wrote:
>>> 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.
>>>
>>
>> The trick here is that ut_asserteq() takes two arguments and casts both as
>> unsigned int, hence why it complains.
>>
>> If Coverity is happy, I'm happy :)
>>
>> This looks fine to me, so:
>>
>> Reviewed-by: Quentin Schulz <quentin.schulz at cherry.de>
>
> Yes, were you able to push this through coverity on your own?
>
I understand this could be misunderstood as I ran Coverity on it, but no
I didn't. Read as "If this makes Coverity happy, then I'm happy".
Sorry for the confusion.
Cheers,
Quentin
More information about the U-Boot
mailing list