app/compress-perf: fix memory deallocation
authorAdam Dybkowski <adamx.dybkowski@intel.com>
Tue, 6 Aug 2019 09:40:53 +0000 (11:40 +0200)
committerAkhil Goyal <akhil.goyal@nxp.com>
Wed, 9 Oct 2019 09:50:12 +0000 (11:50 +0200)
This patch fixes the memory deallocation issue which happened
after unsuccessful allocation (e.g. due to the out of memory)
and produced the segmentation fault.

Fixes: 424dd6c8c1 ("app/compress-perf: add weak functions for multicore test")
Cc: stable@dpdk.org
Signed-off-by: Adam Dybkowski <adamx.dybkowski@intel.com>
Acked-by: Artur Trybula <arturx.trybula@intel.com>
app/test-compress-perf/comp_perf_test_common.c

index 6edc40f..7b26734 100644 (file)
@@ -81,10 +81,13 @@ comp_perf_free_memory(struct cperf_mem_resources *mem)
 {
        uint32_t i;
 
-       for (i = 0; i < mem->total_bufs; i++) {
-               rte_pktmbuf_free(mem->comp_bufs[i]);
-               rte_pktmbuf_free(mem->decomp_bufs[i]);
-       }
+       if (mem->decomp_bufs != NULL)
+               for (i = 0; i < mem->total_bufs; i++)
+                       rte_pktmbuf_free(mem->decomp_bufs[i]);
+
+       if (mem->comp_bufs != NULL)
+               for (i = 0; i < mem->total_bufs; i++)
+                       rte_pktmbuf_free(mem->comp_bufs[i]);
 
        rte_free(mem->decomp_bufs);
        rte_free(mem->comp_bufs);