From 47dc18d2d3e9ce1d984d78d0340c6496871a6fc6 Mon Sep 17 00:00:00 2001
From: Daniel Verkamp <daniel.verkamp@intel.com>
Date: Thu, 30 Nov 2017 09:28:45 -0700
Subject: [PATCH] bdev/null: free Null bdev names on shutdown

Use the existing bdev_null_destruct() function instead of open-coding
the bdev cleanup; the bdev_null_finish() path neglected to free the bdev
name.

Change-Id: Ie74be39b0475674f70c3999bbe2f21caab2e2e20
Signed-off-by: Daniel Verkamp <daniel.verkamp@intel.com>
Reviewed-on: https://review.gerrithub.io/389867
Tested-by: SPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
---
 lib/bdev/null/bdev_null.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/lib/bdev/null/bdev_null.c b/lib/bdev/null/bdev_null.c
index c66c86c5f..39520b0c0 100644
--- a/lib/bdev/null/bdev_null.c
+++ b/lib/bdev/null/bdev_null.c
@@ -267,8 +267,7 @@ bdev_null_finish(void)
 	struct null_bdev *bdev, *tmp;
 
 	TAILQ_FOREACH_SAFE(bdev, &g_null_bdev_head, tailq, tmp) {
-		TAILQ_REMOVE(&g_null_bdev_head, bdev, tailq);
-		spdk_dma_free(bdev);
+		bdev_null_destruct(bdev);
 	}
 }