From 74f44dd807ab9fcf5975915c00cb872ba88fa7c6 Mon Sep 17 00:00:00 2001 From: Bernard Iremonger Date: Fri, 28 Jun 2019 14:29:21 +0100 Subject: [PATCH] test/ipsec: fix tear down Set freed cop pointers to NULL Set other freed pointers to NULL instead of 0 Fixes: 05fe65eb66b2 ("test/ipsec: introduce functional test") Cc: stable@dpdk.org Signed-off-by: Bernard Iremonger Acked-by: Akhil Goyal --- app/test/test_ipsec.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/app/test/test_ipsec.c b/app/test/test_ipsec.c index 0667dd021e..df878846e5 100644 --- a/app/test/test_ipsec.c +++ b/app/test/test_ipsec.c @@ -452,8 +452,10 @@ ut_teardown(void) for (i = 0; i < BURST_SIZE; i++) { /* free crypto operation structure */ - if (ut_params->cop[i]) + if (ut_params->cop[i]) { rte_crypto_op_free(ut_params->cop[i]); + ut_params->cop[i] = NULL; + } /* * free mbuf - both obuf and ibuf are usually the same, @@ -463,17 +465,17 @@ ut_teardown(void) if (ut_params->obuf[i]) { rte_pktmbuf_free(ut_params->obuf[i]); if (ut_params->ibuf[i] == ut_params->obuf[i]) - ut_params->ibuf[i] = 0; - ut_params->obuf[i] = 0; + ut_params->ibuf[i] = NULL; + ut_params->obuf[i] = NULL; } if (ut_params->ibuf[i]) { rte_pktmbuf_free(ut_params->ibuf[i]); - ut_params->ibuf[i] = 0; + ut_params->ibuf[i] = NULL; } if (ut_params->testbuf[i]) { rte_pktmbuf_free(ut_params->testbuf[i]); - ut_params->testbuf[i] = 0; + ut_params->testbuf[i] = NULL; } } -- 2.20.1