From 0b1799cd98d048cc797d41c466149fa5ab539c11 Mon Sep 17 00:00:00 2001 From: Seth Howell Date: Tue, 28 Jul 2020 08:59:29 -0700 Subject: [PATCH] nvme/transport: addd assert for transport. Silences a KW error. Signed-off-by: Seth Howell Change-Id: Ifd8d6088a22de7c230d48751be2b3991d0649778 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/3553 Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins Reviewed-by: Changpeng Liu Reviewed-by: Aleksey Marchuk --- lib/nvme/nvme_transport.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/nvme/nvme_transport.c b/lib/nvme/nvme_transport.c index 1f47a025b..76efd5966 100644 --- a/lib/nvme/nvme_transport.c +++ b/lib/nvme/nvme_transport.c @@ -280,6 +280,8 @@ nvme_transport_ctrlr_delete_io_qpair(struct spdk_nvme_ctrlr *ctrlr, struct spdk_ { const struct spdk_nvme_transport *transport = nvme_get_transport(ctrlr->trid.trstring); + assert(transport != NULL); + /* Do not rely on qpair->transport. For multi-process cases, a foreign process may delete * the IO qpair, in which case the transport object would be invalid (each process has their * own unique transport objects since they contain function pointers). So we look up the