From 2a6a49125058148b086dd906c8af4952a52c0aa2 Mon Sep 17 00:00:00 2001 From: Seth Howell Date: Wed, 22 Apr 2020 16:53:11 -0700 Subject: [PATCH] bdev/nvme: add an assert for nvme_bdev_ctrlr when populating namespaces. This is always going to be true because of the context we are in when calling this function, but add an assert just to be safe. Signed-off-by: Seth Howell Change-Id: Ifbd075865a191529eb9e656c360f182dba8a85bc Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/1985 Community-CI: Broadcom CI Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins Reviewed-by: Shuhei Matsumoto Reviewed-by: Tomasz Zawadzki Reviewed-by: Aleksey Marchuk Reviewed-by: Jim Harris --- module/bdev/nvme/bdev_nvme.c | 1 + 1 file changed, 1 insertion(+) diff --git a/module/bdev/nvme/bdev_nvme.c b/module/bdev/nvme/bdev_nvme.c index 6b0bf4203..465060707 100644 --- a/module/bdev/nvme/bdev_nvme.c +++ b/module/bdev/nvme/bdev_nvme.c @@ -1540,6 +1540,7 @@ nvme_ctrlr_populate_namespaces_done(struct nvme_async_probe_ctx *ctx) size_t j; nvme_bdev_ctrlr = nvme_bdev_ctrlr_get(&ctx->trid); + assert(nvme_bdev_ctrlr != NULL); /* * Report the new bdevs that were created in this call.