[PATCH] test: remove unneeded null check before free

Marat Khalili qm2k21 at gmail.com
Sat Oct 3 09:25:40 CEST 2026


On 02/10/2026 18:17, Stephen Hemminger wrote:
> Found by devtools/null_free.cocci.

nit: should the commit message quote specific coccinelle output being 
fixed?

> Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
> ---
>   app/test/test_cksum.c | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/app/test/test_cksum.c b/app/test/test_cksum.c
> index 73f2511731..d94539820a 100644
> --- a/app/test/test_cksum.c
> +++ b/app/test/test_cksum.c
> @@ -320,8 +320,7 @@ test_l4_cksum_mbuf_chain(struct rte_mempool *pktmbuf_pool, const char *pktdata,
>   	return 0;
>   
>   fail:
> -	if (m_hdr)
> -		rte_pktmbuf_free(m_hdr);
> +	rte_pktmbuf_free(m_hdr);
>   
>   	return -1;
>   }

Acked-by: Marat Khalili <qm2k21 at gmail.com>



More information about the dev mailing list