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>

Reply via email to