Hi David, > -----Original Message----- > From: Coyle, David <[email protected]> > Sent: Thursday, July 16, 2020 4:32 PM > To: [email protected]; Doherty, Declan <[email protected]>; De > Lara Guarch, Pablo <[email protected]>; Trahe, Fiona > <[email protected]> > Cc: [email protected]; Ryan, Brendan <[email protected]>; O'loingsigh, > Mairtin <[email protected]>; Coyle, David <[email protected]> > Subject: [PATCH v1 2/2] crypto/aesni_mb: improve DOCSIS session creation > > This patch improves the DOCSIS session creation as follows: > - it validates the security action type as well as the protocol before > creating a session and now does this validation before allocating the > session from the mempool > > Fixes: fda5216fba55 ("crypto/aesni_mb: support DOCSIS protocol") > > Signed-off-by: David Coyle <[email protected]>
Nice, this is actually fixing a potential memory leak, so you could mention this in the commit message/title. Apart from this: Acked-by: Pablo de Lara <[email protected]>
