From 9c310f7c5ee5e32db349a70120ac82523afb2c51 Mon Sep 17 00:00:00 2001
From: Michael Scherer <misc@redhat.com>
Date: Wed, 22 Feb 2017 18:04:59 +0100
Subject: Remove double free for disk_layout

Since disk_layout is freed after the jump to err, we
do not need to free it again before the goto.

Found by coverity scan.

Change-Id: Ie0c0262f6b95c51c61a59faefbca70352bf1e604
BUG: 789278
Signed-off-by: Michael Scherer <misc@redhat.com>
Reviewed-on: https://review.gluster.org/16720
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
Tested-by: Michael Scherer <misc@fedoraproject.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
---
 xlators/cluster/dht/src/dht-selfheal.c | 2 --
 1 file changed, 2 deletions(-)

(limited to 'xlators/cluster/dht/src/dht-selfheal.c')

diff --git a/xlators/cluster/dht/src/dht-selfheal.c b/xlators/cluster/dht/src/dht-selfheal.c
index db69f8b433c..0838a627521 100644
--- a/xlators/cluster/dht/src/dht-selfheal.c
+++ b/xlators/cluster/dht/src/dht-selfheal.c
@@ -2523,8 +2523,6 @@ dht_update_commit_hash_for_layout_resume (call_frame_t *frame, void *cookie,
                                 " dictionary,", local->loc.path,
                                 conf->local_subvols[i]->name);
 
-                        GF_FREE (disk_layout);
-
                         goto err;
                 }
                 disk_layout = NULL;
-- 
cgit