summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGaurav Kumar Garg <garg.gaurav52@gmail.com>2016-03-10 07:51:48 +0530
committerJeff Darcy <jdarcy@redhat.com>2016-03-10 05:54:58 -0800
commita007fdf549260d0b146184fa85ca7029560db8c5 (patch)
treeb29b90149c581d30bf9beb86b2a61c0e0980aa6a
parent6332eb773914ffd92aa78376c11908ddecb99d6e (diff)
glusterd: fixing few memory leak in glusterd
While freeing memory currently glusterd is not freeing correct memory. this might result in some serious situation. With this fix glusterd will free correct memory location. Change-Id: Ide9c33a2ec5822b560e9e2dfcb6a0b442fc97047 BUG: 1287517 Signed-off-by: Gaurav Kumar Garg <ggarg@redhat.com> Reviewed-on: http://review.gluster.org/13660 Smoke: Gluster Build System <jenkins@build.gluster.com> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> Reviewed-by: Vijay Bellur <vbellur@redhat.com> CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-rpc-ops.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-rpc-ops.c b/xlators/mgmt/glusterd/src/glusterd-rpc-ops.c
index e50158e11a7..39941d17b8a 100644
--- a/xlators/mgmt/glusterd/src/glusterd-rpc-ops.c
+++ b/xlators/mgmt/glusterd/src/glusterd-rpc-ops.c
@@ -1717,7 +1717,7 @@ out:
GF_FREE (req.friends.friends_val);
if (ret && dummy_frame)
- STACK_DESTROY (frame->root);
+ STACK_DESTROY (dummy_frame->root);
gf_msg_debug ("glusterd", 0, "Returning %d", ret);
return ret;
@@ -1756,7 +1756,7 @@ out:
gf_msg_debug ("glusterd", 0, "Returning %d", ret);
if (ret && dummy_frame)
- STACK_DESTROY (frame->root);
+ STACK_DESTROY (dummy_frame->root);
return ret;
}
@@ -1945,7 +1945,7 @@ out:
gf_msg_debug (this ? this->name : "glusterd", 0, "Returning %d", ret);
if (ret && dummy_frame)
- STACK_DESTROY (frame->root);
+ STACK_DESTROY (dummy_frame->root);
return ret;
}