From 60a208c004c8fce2bc8235dbf52f7a6b128600e7 Mon Sep 17 00:00:00 2001
From: Seth Howell <seth.howell@intel.com>
Date: Mon, 10 Sep 2018 12:48:56 -0700
Subject: [PATCH] iscsi: free lun after bdev_destruct_done returns

This prevents us from using the bdev embedded in the lun after it has
been freed.

Change-Id: I780cf3eccca05a58d3461366fec024be42b8ff74
Signed-off-by: Seth Howell <seth.howell@intel.com>
Reviewed-on: https://review.gerrithub.io/425174
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
---
 lib/bdev/iscsi/bdev_iscsi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/bdev/iscsi/bdev_iscsi.c b/lib/bdev/iscsi/bdev_iscsi.c
index d4611359b..528337f55 100644
--- a/lib/bdev/iscsi/bdev_iscsi.c
+++ b/lib/bdev/iscsi/bdev_iscsi.c
@@ -139,9 +139,9 @@ _iscsi_free_lun(void *arg)
 	free(lun->bdev.name);
 	free(lun->url);
 	free(lun->initiator_iqn);
-	free(lun);
 
 	spdk_bdev_destruct_done(&lun->bdev, 0);
+	free(lun);
 }
 
 static void