On 02/01/2016 03:26 PM, Herbert Xu wrote: > On Fri, Jan 29, 2016 at 02:10:09PM +0100, Jerome Marchand wrote: >> __test_aead() reads MAX_IVLEN bytes from template[i].iv, but the >> actual length of the initialisation vector can be shorter. >> The length of the IV is already calculated earlier in the >> function. Let's just reuses that. >> This fix an out-of-bound error detected by KASan. >> >> Signed-off-by: Jerome Marchand <jmarc...@redhat.com> > > This patch creates a new warning that iv_len may be uninitialised.
I see. iv_len is set for each templates. I don't see why we would like to call crypto_aead_ivsize() more than once. Moving the initialization of iv_len out of the loop should solve the warning. > > Please fix this and resubmit. Will do. Jerome > > Thanks, >
signature.asc
Description: OpenPGP digital signature
