Commit 09d3e4c9 authored by Daniel Verkamp's avatar Daniel Verkamp
Browse files

test/nvme: use SPDK_CU_ASSERT_FATAL() 



Use SPDK_CU_ASSERT_FATAL instead of CU_ASSERT_FATAL so static analyzers
recognize that g_request cannot be NULL in the following lines.

Change-Id: Ie7ab3bd34a177bea0d565441014e8db12be8bb01
Signed-off-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
parent 53cb5d8f

test/lib/nvme/unit/nvme_ns_cmd_c/nvme_ns_cmd_ut.c

+2 −2
Original line number Diff line number Diff line
@@ -559,7 +559,7 @@ test_io_flags(void) 
	rc = spdk_nvme_ns_cmd_read(&ns, &qpair, payload, lba, lba_count, NULL, NULL,

				   SPDK_NVME_IO_FLAGS_FORCE_UNIT_ACCESS);

	CU_ASSERT(rc == 0);

	CU_ASSERT_FATAL(g_request != NULL);

	SPDK_CU_ASSERT_FATAL(g_request != NULL);

	CU_ASSERT((g_request->cmd.cdw12 & SPDK_NVME_IO_FLAGS_FORCE_UNIT_ACCESS) != 0);

	CU_ASSERT((g_request->cmd.cdw12 & SPDK_NVME_IO_FLAGS_LIMITED_RETRY) == 0);

	nvme_free_request(g_request);
@@ -567,7 +567,7 @@ test_io_flags(void) 
	rc = spdk_nvme_ns_cmd_read(&ns, &qpair, payload, lba, lba_count, NULL, NULL,

				   SPDK_NVME_IO_FLAGS_LIMITED_RETRY);

	CU_ASSERT(rc == 0);

	CU_ASSERT_FATAL(g_request != NULL);

	SPDK_CU_ASSERT_FATAL(g_request != NULL);

	CU_ASSERT((g_request->cmd.cdw12 & SPDK_NVME_IO_FLAGS_FORCE_UNIT_ACCESS) == 0);

	CU_ASSERT((g_request->cmd.cdw12 & SPDK_NVME_IO_FLAGS_LIMITED_RETRY) != 0);

	nvme_free_request(g_request);