From ecdbaa2310e243e83095396942b888267bbdb151 Mon Sep 17 00:00:00 2001 From: Shuhei Matsumoto Date: Wed, 23 Feb 2022 15:04:42 +0900 Subject: [PATCH] bdev/nvme: Call spdk_free() to the object allocated by spdk_malloc() This is a bug fix. free() was called to the object allocated by spdk_malloc(). Hence free(): invalid pointer: 0x00002000146ece00 was printed. This was found during multipath testing. Signed-off-by: Shuhei Matsumoto Change-Id: Icf6aa6dcdda728fef91b3acad7a1f1ee219c27af Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11710 Community-CI: Mellanox Build Bot Community-CI: Broadcom CI Reviewed-by: Tomasz Zawadzki Reviewed-by: Jim Harris Reviewed-by: Ben Walker Reviewed-by: Aleksey Marchuk Tested-by: SPDK CI Jenkins --- module/bdev/nvme/bdev_nvme.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/module/bdev/nvme/bdev_nvme.c b/module/bdev/nvme/bdev_nvme.c index d19838974..d7caf9294 100644 --- a/module/bdev/nvme/bdev_nvme.c +++ b/module/bdev/nvme/bdev_nvme.c @@ -3126,7 +3126,7 @@ bdev_nvme_disable_read_ana_log_page(struct nvme_ctrlr *nvme_ctrlr) { struct nvme_ns *nvme_ns; - free(nvme_ctrlr->ana_log_page); + spdk_free(nvme_ctrlr->ana_log_page); nvme_ctrlr->ana_log_page = NULL; for (nvme_ns = nvme_ctrlr_get_first_active_ns(nvme_ctrlr);