From 3f6a54c1a7dc8d21264f58298cfa61814e45de56 Mon Sep 17 00:00:00 2001
From: Li Feng <lifeng1519@gmail.com>
Date: Fri, 31 Aug 2018 21:39:27 +0800
Subject: [PATCH] bdev/rbd: fix memory leak and spdk_io_device_register fail

io_device is registered, but never release.
It will lead spdk_io_device_register to fail
because allocate new io_device may reuse the same memory address.

Change-Id: I3359ed783c1df43ef382669d9d73d06bfbc1fe2d
Signed-off-by: Li Feng <lifeng1519@gmail.com>
Reviewed-on: https://review.gerrithub.io/424235
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: GangCao <gang.cao@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
---
 lib/bdev/rbd/bdev_rbd.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/lib/bdev/rbd/bdev_rbd.c b/lib/bdev/rbd/bdev_rbd.c
index 4904d4da2..f85778b16 100644
--- a/lib/bdev/rbd/bdev_rbd.c
+++ b/lib/bdev/rbd/bdev_rbd.c
@@ -312,6 +312,8 @@ bdev_rbd_destruct(void *ctx)
 {
 	struct bdev_rbd *rbd = ctx;
 
+	spdk_io_device_unregister(rbd, NULL);
+
 	bdev_rbd_free(rbd);
 	return 0;
 }