From a007fdf549260d0b146184fa85ca7029560db8c5 Mon Sep 17 00:00:00 2001 From: Gaurav Kumar Garg Date: Thu, 10 Mar 2016 07:51:48 +0530 Subject: 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 Reviewed-on: http://review.gluster.org/13660 Smoke: Gluster Build System NetBSD-regression: NetBSD Build System Reviewed-by: Vijay Bellur CentOS-regression: Gluster Build System --- xlators/mgmt/glusterd/src/glusterd-rpc-ops.c | 6 +++--- 1 file 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; } -- cgit