Re: [dpdk-dev] [PATCH 3/3] compress/isal: fix memory leak

["Daly, Lee" <lee.daly@intel.com>]

> -----Original Message-----
> From: De Lara Guarch, Pablo
> Sent: Wednesday, July 11, 2018 7:39 AM
> To: Daly, Lee <lee.daly@intel.com>
> Cc: dev@dpdk.org; De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com>;
> stable@dpdk.org
> Subject: [PATCH 3/3] compress/isal: fix memory leak
> 
> Processed operations ring is created for each queue pair, but it was not being
> freed when the queue pair was released.
> 
> Fixes: b0e23c458a6f ("compress/isal: add queue pair related ops")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> ---
>  drivers/compress/isal/isal_compress_pmd_ops.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/compress/isal/isal_compress_pmd_ops.c
> b/drivers/compress/isal/isal_compress_pmd_ops.c
> index 0738fb9c4..c61acd40c 100644
> --- a/drivers/compress/isal/isal_compress_pmd_ops.c
> +++ b/drivers/compress/isal/isal_compress_pmd_ops.c
> @@ -167,6 +167,9 @@ isal_comp_pmd_qp_release(struct rte_compressdev
> *dev, uint16_t qp_id)
>  	if (qp->state != NULL)
>  		rte_free(qp->state);
> 
> +	if (qp->processed_pkts != NULL)
> +		rte_ring_free(qp->processed_pkts);
> +
>  	rte_free(qp);
>  	dev->data->queue_pairs[qp_id] = NULL;
> 
> --
> 2.14.4
Acked-by: Lee Daly <lee.daly@intel.com>