Commit e3b90fe2 authored by yidong0635's avatar yidong0635 Committed by Darek Stojaczyk
Browse files

bdev: Fix warning about potenial memory leak about histogram. 



Pointer passthrough g_histogram and histogram implicitly ,but the checker
still complains about this, change to spdk_histogram_data_free(histogram)
directly, no need any assert any more.

bdev_ut.c:1486:2: warning: Potential leak of memory pointed to by 'histogram'
        poll_threads();
        ^~~~~~~~~~~~~~
1 warning generated.

Change-Id: I609c1c92643ad03b8e3c62cd8434882e7942efa4
Signed-off-by: default avataryidong0635 <dongx.yi@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/462789


Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent a78b7bb1

test/unit/lib/bdev/mt/bdev.c/bdev_ut.c

+1 −1
Original line number Diff line number Diff line
@@ -1498,7 +1498,7 @@ bdev_histograms_mt(void) 
	CU_ASSERT(g_status == 0);

	CU_ASSERT(g_bdev.bdev.internal.histogram_enabled == false);



	spdk_histogram_data_free(g_histogram);

	spdk_histogram_data_free(histogram);



	/* Tear down the channels */

	set_thread(0);