From 7bf3ac17f7f178a49b87e32e63472d095f94b6cf Mon Sep 17 00:00:00 2001 From: Chunyang Hui Date: Tue, 18 Jun 2019 20:58:50 +0800 Subject: [PATCH] Opal: close dev in nvme_manage free resources when error was triggered in nvme_manage Change-Id: Ibdc020ffb5abd3d614fe85e54788b1818ef5af1e Signed-off-by: Chunyang Hui Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/458342 Tested-by: SPDK CI Jenkins Reviewed-by: Changpeng Liu Reviewed-by: Ben Walker --- examples/nvme/nvme_manage/nvme_manage.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/examples/nvme/nvme_manage/nvme_manage.c b/examples/nvme/nvme_manage/nvme_manage.c index 1c50dc32b..77332f7b0 100644 --- a/examples/nvme/nvme_manage/nvme_manage.c +++ b/examples/nvme/nvme_manage/nvme_manage.c @@ -988,6 +988,7 @@ opal_take_ownership(struct dev *iter) ret = spdk_opal_cmd_take_ownership(iter->opal_dev, passwd_p); if (ret) { printf("Take ownership failure: %d\n", ret); + spdk_opal_close(iter->opal_dev); return; } printf("...\n...\nTake Ownership Success\n"); @@ -1024,6 +1025,7 @@ opal_revert_tper(struct dev *iter) ret = spdk_opal_cmd_revert_tper(iter->opal_dev, passwd_p); if (ret) { printf("Revert TPer failure: %d\n", ret); + spdk_opal_close(iter->opal_dev); return; } printf("...\n...\nRevert TPer Success\n");