From b584c068c6526853f82ccd2a406c3596d7d56be0 Mon Sep 17 00:00:00 2001
From: Nigel Babu <nigelb@redhat.com>
Date: Sat, 18 Feb 2017 14:02:30 +0530
Subject: libglusterfs, dht, locks, glusterd: Coverity fixes

Fix up use after free bugs and dead code

Change-Id: I8f79ed6b5108926c1fac31c147b5ecba79d10785
BUG: 1424905
Signed-off-by: Nigel Babu <nigelb@redhat.com>
Reviewed-on: https://review.gluster.org/16666
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
---
 libglusterfs/src/syncop.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'libglusterfs/src')

diff --git a/libglusterfs/src/syncop.c b/libglusterfs/src/syncop.c
index 00a9b576269..7275f22b099 100644
--- a/libglusterfs/src/syncop.c
+++ b/libglusterfs/src/syncop.c
@@ -831,8 +831,10 @@ syncenv_new (size_t stacksize, int procmin, int procmax)
                 newenv->procs++;
         }
 
-        if (ret != 0)
+        if (ret != 0) {
                 syncenv_destroy (newenv);
+                newenv = NULL;
+        }
 
         return newenv;
 }
-- 
cgit