[PATCH][next] mtd: nand: Fix error handling in nand_prog_page_op

Miquel Raynal miquel.raynal at bootlin.com
Wed Mar 3 09:46:51 GMT 2021


Hi Colin,

Colin King <colin.king at canonical.com> wrote on Wed,  3 Mar 2021
09:42:46 +0000:

> From: Colin Ian King <colin.king at canonical.com>
> 
> The less than zero comparison with status is always false because status
> is a u8. Fix this by using ret as the return check for the call to
> chip->legacy.waitfunc() and checking on this and assigning status to ret
> if it is OK.
> 
> Addresses-Coverity: ("Unsigned compared against 0")
> Fixes: 52f67def97f1 ("mtd: nand: fix error handling in nand_prog_page_op() #1")
> Signed-off-by: Colin Ian King <colin.king at canonical.com>

Thanks for the fix, but this has been handled just an hour ago :)

Cheers,
Miquèl



More information about the linux-mtd mailing list