crypto: caam - fix unchecked return value error
authorGaurav Jain <gaurav.jain@nxp.com>
Tue, 8 Aug 2023 10:55:25 +0000 (12:55 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Fri, 18 Aug 2023 09:01:08 +0000 (17:01 +0800)
error:
Unchecked return value (CHECKED_RETURN)
check_return: Calling sg_miter_next without checking return value

fix:
added check if(!sg_miter_next)

Fixes: 8a2a0dd35f2e ("crypto: caam - strip input zeros from RSA input buffer")
Signed-off-by: Gaurav Jain <gaurav.jain@nxp.com>
Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@nxp.com>
Reviewed-by: Gaurav Jain <gaurav.jain@nxp.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/caam/caampkc.c

index 72afc24..7e08af7 100644 (file)
@@ -225,7 +225,9 @@ static int caam_rsa_count_leading_zeros(struct scatterlist *sgl,
                if (len && *buff)
                        break;
 
-               sg_miter_next(&miter);
+               if (!sg_miter_next(&miter))
+                       break;
+
                buff = miter.addr;
                len = miter.length;