Hi David, > -----Original Message----- > From: Coyle, David <david.co...@intel.com> > Sent: Thursday, July 16, 2020 4:32 PM > To: akhil.go...@nxp.com; Doherty, Declan <declan.dohe...@intel.com>; De > Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; Trahe, Fiona > <fiona.tr...@intel.com> > Cc: dev@dpdk.org; Ryan, Brendan <brendan.r...@intel.com>; O'loingsigh, > Mairtin <mairtin.oloings...@intel.com>; Coyle, David <david.co...@intel.com> > 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 <david.co...@intel.com>
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 <pablo.de.lara.gua...@intel.com>