[PATCH] test: cmd: part: fix integer handling in setup_gpt_partitions()

Tom Rini trini at konsulko.com
Fri Jun 26 23:58:04 CEST 2026


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?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20260626/650dce5a/attachment.sig>


More information about the U-Boot mailing list