From 76c501834e3cfd3420d959e38a68aeb72017adc6 Mon Sep 17 00:00:00 2001
From: Ziye Yang <ziye.yang@intel.com>
Date: Wed, 7 Mar 2018 13:31:06 +0800
Subject: [PATCH] lib/nvmf: set sgroup->channels[nsid] = NULL

We should add this state after free the io channel.
To make sure that we will not touch the wild pointer.

Change-Id: I570f649fe3ddffaea430125b40c6b6b020a05222
Signed-off-by: Ziye Yang <ziye.yang@intel.com>
Reviewed-on: https://review.gerrithub.io/402999
Tested-by: SPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: Daniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
---
 lib/nvmf/nvmf.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/nvmf/nvmf.c b/lib/nvmf/nvmf.c
index 837d815e5..009557e23 100644
--- a/lib/nvmf/nvmf.c
+++ b/lib/nvmf/nvmf.c
@@ -474,6 +474,7 @@ poll_group_update_subsystem(struct spdk_nvmf_poll_group *group,
 		for (i = new_num_channels; i < old_num_channels; i++) {
 			if (sgroup->channels[i]) {
 				spdk_put_io_channel(sgroup->channels[i]);
+				sgroup->channels[i] = NULL;
 			}
 		}