d1681e
From 05c63817ac715fdcf9065568d796b93d97c0be7f Mon Sep 17 00:00:00 2001
d1681e
From: Jiffin Tony Thottan <jthottan@redhat.com>
d1681e
Date: Tue, 2 May 2017 14:06:00 +0530
d1681e
Subject: [PATCH 62/74] glusterd/ganesha : add proper NULL check in
d1681e
 manage_export_config
d1681e
d1681e
Change-Id: I872b2b6b027f04e61f60ad85588f50e1ef2f988c
d1681e
Signed-off-by: Jiffin Tony Thottan <jthottan@redhat.com>
d1681e
Reviewed-on: https://review.gluster.org/17150
d1681e
Smoke: Gluster Build System <jenkins@build.gluster.org>
d1681e
Reviewed-by: soumya k <skoduri@redhat.com>
d1681e
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
d1681e
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
d1681e
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
d1681e
---
d1681e
 xlators/mgmt/glusterd/src/glusterd-ganesha.c | 2 +-
d1681e
 1 file changed, 1 insertion(+), 1 deletion(-)
d1681e
d1681e
diff --git a/xlators/mgmt/glusterd/src/glusterd-ganesha.c b/xlators/mgmt/glusterd/src/glusterd-ganesha.c
d1681e
index cd591df..7ba25ee 100644
d1681e
--- a/xlators/mgmt/glusterd/src/glusterd-ganesha.c
d1681e
+++ b/xlators/mgmt/glusterd/src/glusterd-ganesha.c
d1681e
@@ -454,7 +454,7 @@ manage_export_config (char *volname, char *value, char **op_errstr)
d1681e
                          CONFDIR, value, volname, NULL);
d1681e
         ret = runner_run(&runner);
d1681e
 
d1681e
-        if (ret && !(*op_errstr))
d1681e
+        if (ret && op_errstr)
d1681e
                 gf_asprintf (op_errstr, "Failed to create"
d1681e
                             " NFS-Ganesha export config file.");
d1681e
 
d1681e
-- 
d1681e
1.8.3.1
d1681e