ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ut/compress: fix compress UT

Browse Source 
Remove code that was freeing memory in the code under test. It was
no longer needed following a refactor of the init code and was
causing a double free.

Change-Id: I99018587ac12e01d4fe950e0718c23790ce7ce41
Signed-off-by: paul luse <paul.e.luse@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/465966
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
This commit is contained in:
paul luse 2019-08-21 12:56:46 -04:00 committed by Jim Harris
parent 68e7da44ec
commit c8e034b092
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
test/unit/lib/bdev/compress.c/compress_ut.c
View File

@ -736,8 +736,6 @@ test_initdrivers(void)
{ {
int rc; int rc;
static struct rte_mempool *orig_mbuf_mp; static struct rte_mempool *orig_mbuf_mp;
struct comp_device_qp *dev_qp;
struct comp_device_qp *tmp_qp;
orig_mbuf_mp = g_mbuf_mp; orig_mbuf_mp = g_mbuf_mp;
g_mbuf_mp = NULL; g_mbuf_mp = NULL;
@ -820,12 +818,6 @@ test_initdrivers(void)
rc = vbdev_init_compress_drivers(); rc = vbdev_init_compress_drivers();
CU_ASSERT(rc == 0); CU_ASSERT(rc == 0);
TAILQ_FOREACH_SAFE(dev_qp, &g_comp_device_qp, link, tmp_qp) {
TAILQ_REMOVE(&g_comp_device_qp, dev_qp, link);
free(dev_qp->device);
free(dev_qp);
}
g_mbuf_mp = orig_mbuf_mp; g_mbuf_mp = orig_mbuf_mp;
} }