Commit 0a4a217c authored by Jim Harris's avatar Jim Harris
Browse files

test/bdev: fix enomem_multi_bdev unit test 



Latest scan-build finds a legitimate issue here - although
it's not clear why it was only found when testing against
a seemingly unrelated patch...

Signed-off-by: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: Ic981f6f403c638b0b2d454c0be9f7fbe145171d9

Reviewed-on: https://review.gerrithub.io/424887


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
parent 9ad05b31

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

+2 −1
Original line number Diff line number Diff line
@@ -930,7 +930,7 @@ enomem_multi_bdev(void) 
	enum spdk_bdev_io_status status[IO_ARRAY_SIZE];

	uint32_t i;

	struct ut_bdev *second_bdev;

	struct spdk_bdev_desc *second_desc;

	struct spdk_bdev_desc *second_desc = NULL;

	struct spdk_bdev_channel *second_bdev_ch;

	struct spdk_io_channel *second_ch;

	int rc;
@@ -942,6 +942,7 @@ enomem_multi_bdev(void) 
	SPDK_CU_ASSERT_FATAL(second_bdev != NULL);

	register_bdev(second_bdev, "ut_bdev2", g_bdev.io_target);

	spdk_bdev_open(&second_bdev->bdev, true, NULL, NULL, &second_desc);

	SPDK_CU_ASSERT_FATAL(second_desc != NULL);



	set_thread(0);

	io_ch = spdk_bdev_get_io_channel(g_desc);