Commit 9e9cd51f authored by Daniel Verkamp's avatar Daniel Verkamp
Browse files

test/nvme_ns_cmd: zero-initialize ctrlr options 



A recently-added test now uses a controller option value that was
previously uninitialized.  Set them to all zeroes to avoid a Valgrind
warning about using uninitialized data.

Change-Id: I4454b781b7aec27e6d4f4dd8ff5cb059270551e9
Fixes: fbf06aca ("nvme: fail when request is too long")
Signed-off-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-on: https://review.gerrithub.io/413017


Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
parent 98d140d2

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

+1 −0
Original line number Diff line number Diff line
@@ -212,6 +212,7 @@ prepare_for_test(struct spdk_nvme_ns *ns, struct spdk_nvme_ctrlr *ctrlr, 
	ctrlr->flags = 0;

	ctrlr->min_page_size = 4096;

	ctrlr->page_size = 4096;

	memset(&ctrlr->opts, 0, sizeof(ctrlr->opts));

	memset(ns, 0, sizeof(*ns));

	ns->ctrlr = ctrlr;

	ns->sector_size = sector_size;