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 <[email protected]>
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
