Hi Piotr, > -----Original Message----- > From: Zhang, Roy Fan <[email protected]> > Sent: Monday, March 7, 2022 4:27 PM > To: Bronowski, PiotrX <[email protected]>; [email protected] > Cc: [email protected]; [email protected]; Yigit, Ferruh > <[email protected]>; Doherty, Declan <[email protected]>; > [email protected] > Subject: RE: [PATCH v2] crypto/ipsec_mb: fix usage of untrusted value > > > -----Original Message----- > > From: Bronowski, PiotrX <[email protected]> > > Sent: Monday, March 7, 2022 3:33 PM > > To: [email protected] > > Cc: Zhang, Roy Fan <[email protected]>; [email protected]; > > [email protected]; Yigit, Ferruh <[email protected]>; Doherty, > Declan > > <[email protected]>; Bronowski, PiotrX > > <[email protected]>; [email protected] > > Subject: [PATCH v2] crypto/ipsec_mb: fix usage of untrusted value > > > > This patch removes coverity defect CID 375828: > > Untrusted value as argument (TAINTED_SCALAR) > > > > Coverity issue: CID 375828 > > Fixes: 918fd2f1466b ("crypto/ipsec_mb: move aesni_mb PMD") > > > > Signed-off-by: Piotr Bronowski <[email protected]> > > > > Cc: [email protected] > > > > --- > > v2: use a different logic to check digest length > > --- > Acked-by: Fan Zhang <[email protected]>
Sorry I missed a point in your change and thanks for Ciara pointing this out. You are changing the gen_digest_size to 64 which is wrong. Please send v3. Also instead of ack - Nack this patch.
