From b8d0691ff3e350c5a3f45eeec421f6789c30b3ba Mon Sep 17 00:00:00 2001
From: GangCao <gang.cao@intel.com>
Date: Fri, 6 Dec 2019 13:48:29 -0500
Subject: [PATCH] nvmf: correct the return code when parsing the subsystem

If succeeded, return 0. Otherwise, return -1

Change-Id: I7c966e09a5efd13a3ddb72ffaaeee4a9d8ca31b0
Signed-off-by: GangCao <gang.cao@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/476983
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: Alexey Marchuk <alexeymar@mellanox.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Seth Howell <seth.howell@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Broadcom SPDK FC-NVMe CI <spdk-ci.pdl@broadcom.com>
---
 module/event/subsystems/nvmf/conf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/module/event/subsystems/nvmf/conf.c b/module/event/subsystems/nvmf/conf.c
index ac030860a..f686f7c23 100644
--- a/module/event/subsystems/nvmf/conf.c
+++ b/module/event/subsystems/nvmf/conf.c
@@ -496,7 +496,7 @@ spdk_nvmf_parse_subsystem(struct spdk_conf_section *sp)
 	spdk_nvmf_subsystem_set_allow_any_host(subsystem, allow_any_host);
 
 done:
-	return (subsystem != NULL);
+	return (subsystem != NULL) ? 0 : -1;
 }
 
 static int