nvmf: add an assert for the obtained sgroup

Change-Id: Ie00137083143722e9df50fa5a8e4d37000dfe29c Signed-off-by: GangCao <[email protected]> Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/8739 Reviewed-by: Xiaodong Liu <[email protected]> Reviewed-by: Shuhei Matsumoto <[email protected]> Reviewed-by: <[email protected]> Reviewed-by: Aleksey Marchuk <[email protected]> Community-CI: Broadcom CI <[email protected]> Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins <[email protected]>
tmakatos · Jul 13, 2021 · bd4aec1 · bd4aec1
1 parent 5993036
commit bd4aec1
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/lib/nvmf/ctrlr.c b/lib/nvmf/ctrlr.c
@@ -811,6 +811,7 @@ retry_connect(void *arg)
 	int rc;
 
 	sgroup = nvmf_subsystem_pg_from_connect_cmd(req);
+	assert(sgroup != NULL);
 	sgroup->mgmt_io_outstanding++;
 	spdk_poller_unregister(&req->poller);
 	rc = nvmf_ctrlr_cmd_connect(req);
@@ -861,6 +862,7 @@ nvmf_ctrlr_cmd_connect(struct spdk_nvmf_request *req)
 		 * subsystem waiting for this command to complete before unpausing.
 		 */
 		sgroup = nvmf_subsystem_pg_from_connect_cmd(req);
+		assert(sgroup != NULL);
 		sgroup->mgmt_io_outstanding--;
 		SPDK_DEBUGLOG(nvmf, "Subsystem '%s' is not ready for connect, retrying...\n", subsystem->subnqn);
 		req->poller = SPDK_POLLER_REGISTER(retry_connect, req, 100);