diff options
| author | Gaurav Kumar Garg <garg.gaurav52@gmail.com> | 2016-03-10 07:51:48 +0530 |
|---|---|---|
| committer | Jeff Darcy <jdarcy@redhat.com> | 2016-03-10 05:54:58 -0800 |
| commit | a007fdf549260d0b146184fa85ca7029560db8c5 (patch) | |
| tree | b29b90149c581d30bf9beb86b2a61c0e0980aa6a | |
| parent | 6332eb773914ffd92aa78376c11908ddecb99d6e (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.c | 6 |
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; } |