diff options
| author | Michael Scherer <misc@redhat.com> | 2017-02-22 18:04:59 +0100 |
|---|---|---|
| committer | Jeff Darcy <jdarcy@redhat.com> | 2017-02-27 18:02:30 -0500 |
| commit | 9c310f7c5ee5e32db349a70120ac82523afb2c51 (patch) | |
| tree | 7f69db8c194efe507c450cb19dd2400a61c4023c | |
| parent | 1e3538baab7abc29ac329c78182b62558da56d98 (diff) | |
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>
| -rw-r--r-- | xlators/cluster/dht/src/dht-selfheal.c | 2 |
1 files changed, 0 insertions, 2 deletions
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; |