Reviewed-by: Gaurav Jain <gaurav.jain@xxxxxxx> > -----Original Message----- > From: Meenakshi Aggarwal <meenakshi.aggarwal@xxxxxxx> > Sent: Tuesday, August 8, 2023 4:25 PM > To: Horia Geanta <horia.geanta@xxxxxxx>; Varun Sethi <V.Sethi@xxxxxxx>; > Pankaj Gupta <pankaj.gupta@xxxxxxx>; Gaurav Jain <gaurav.jain@xxxxxxx>; > herbert@xxxxxxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx; linux- > crypto@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Cc: Meenakshi Aggarwal <meenakshi.aggarwal@xxxxxxx> > Subject: [PATCH] crypto: caam: fix unchecked return value error > > From: Gaurav Jain <gaurav.jain@xxxxxxx> > > 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@xxxxxxx> > Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@xxxxxxx> > --- > drivers/crypto/caam/caampkc.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/crypto/caam/caampkc.c b/drivers/crypto/caam/caampkc.c > index 72afc249d42f..7e08af751e4e 100644 > --- a/drivers/crypto/caam/caampkc.c > +++ b/drivers/crypto/caam/caampkc.c > @@ -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; > > -- > 2.25.1