Commit 50cc397d authored by Daniel Verkamp's avatar Daniel Verkamp
Browse files

nvme: simplify nvme_ctrlr_destruct qpair loop 



Fixes a scan-build warning about using qpairs after they have been
freed.

Change-Id: I263eabd6b784acf540c66136965f7705ef110a78
Signed-off-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
parent 5639b965

lib/nvme/nvme_ctrlr.c

+2 −2
Original line number Diff line number Diff line
@@ -1359,9 +1359,9 @@ nvme_ctrlr_init_cap(struct spdk_nvme_ctrlr *ctrlr, const union spdk_nvme_cap_reg 
void

nvme_ctrlr_destruct(struct spdk_nvme_ctrlr *ctrlr)

{

	while (!TAILQ_EMPTY(&ctrlr->active_io_qpairs)) {

		struct spdk_nvme_qpair *qpair = TAILQ_FIRST(&ctrlr->active_io_qpairs);

	struct spdk_nvme_qpair *qpair, *tmp;



	TAILQ_FOREACH_SAFE(qpair, &ctrlr->active_io_qpairs, tailq, tmp) {

		spdk_nvme_ctrlr_free_io_qpair(qpair);

	}