From e69baea132b86ec6426379020f0e90074da8559c Mon Sep 17 00:00:00 2001 From: Tomasz Kulasek Date: Fri, 31 Jan 2020 17:33:00 +0100 Subject: [PATCH] lib/nvme: fix wrong status argument in spdk_nvme_ctrlr_format For nvme_ctrlr_cmd_format command status should be used as nvme_completion_poll_cb callback argument instead of pointer to local variable. Change-Id: Id65cb395d137c4e907c1ef019b131e8822ddfe34 Signed-off-by: Tomasz Kulasek Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/483513 Tested-by: SPDK CI Jenkins Community-CI: SPDK CI Jenkins Reviewed-by: Paul Luse Reviewed-by: Shuhei Matsumoto Reviewed-by: Alexey Marchuk --- lib/nvme/nvme_ctrlr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/nvme/nvme_ctrlr.c b/lib/nvme/nvme_ctrlr.c index 46b4839ee..0079a7473 100644 --- a/lib/nvme/nvme_ctrlr.c +++ b/lib/nvme/nvme_ctrlr.c @@ -3069,7 +3069,7 @@ spdk_nvme_ctrlr_format(struct spdk_nvme_ctrlr *ctrlr, uint32_t nsid, } res = nvme_ctrlr_cmd_format(ctrlr, nsid, format, nvme_completion_poll_cb, - &status); + status); if (res) { free(status); return res;