andy/glusterfs
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

glusterd: fixing few memory leak in glusterd

Browse Source 
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>
This commit is contained in:
Gaurav Kumar Garg 2016-03-10 07:51:48 +05:30 committed by Jeff Darcy
parent 6332eb7739
commit a007fdf549
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
xlators/mgmt/glusterd/src/glusterd-rpc-ops.c
View File

@ -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;
}