From 9cb4e555407fab1968a644c20f738bf949188faf Mon Sep 17 00:00:00 2001 From: Ben Walker Date: Thu, 21 Oct 2021 13:16:15 -0700 Subject: [PATCH] test/nvme: Fix buffer zeroing math This meant to zero the entire active namespace list. Signed-off-by: Ben Walker Change-Id: I2da2293b53acd57d3480cf93b052eb1520de35d4 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/10028 Community-CI: Broadcom CI Community-CI: Mellanox Build Bot Reviewed-by: Changpeng Liu Reviewed-by: Shuhei Matsumoto Reviewed-by: Aleksey Marchuk Reviewed-by: Jim Harris Tested-by: SPDK CI Jenkins --- test/unit/lib/nvme/nvme_ctrlr.c/nvme_ctrlr_ut.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/unit/lib/nvme/nvme_ctrlr.c/nvme_ctrlr_ut.c b/test/unit/lib/nvme/nvme_ctrlr.c/nvme_ctrlr_ut.c index 33d1aaa74..89fcdee30 100644 --- a/test/unit/lib/nvme/nvme_ctrlr.c/nvme_ctrlr_ut.c +++ b/test/unit/lib/nvme/nvme_ctrlr.c/nvme_ctrlr_ut.c @@ -2127,7 +2127,7 @@ test_nvme_ctrlr_test_active_ns(void) nsid = spdk_nvme_ctrlr_get_next_active_ns(&ctrlr, nsid); CU_ASSERT(nsid == 0); - memset(ctrlr.active_ns_list, 0, ctrlr.num_ns); + memset(ctrlr.active_ns_list, 0, sizeof(uint32_t) * ctrlr.num_ns); for (nsid = 0; nsid < ctrlr.num_ns; nsid++) { ctrlr.active_ns_list[nsid] = nsid + 1; }