path: root/glusterfsd
diff options
authorKaleb S. KEITHLEY <>2013-11-26 14:23:08 -0500
committerVijay Bellur <>2013-11-30 10:12:42 -0800
commit72f733a64abeffee23fb87a3cb397baea1dc22a4 (patch)
tree09ab2f44a02b3b11ec6b93badc8d96e9d9b1750c /glusterfsd
parentbca9eab359710eb3b826c6441126e2e56f774df5 (diff)
glusterfsd: fix small memory leaks in glusterfsd-mgmt.c
E.g. In glusterfs_volfile_fetch(), req.xdata.xdata_val is allocated in dict_allocate_and_serialize() but not freed after mgmt_submit_request(). A survey of dict_allocate_and_serialize/_submit_request in glusterfsd-mgmt.c shows no consistent pattern of freeing the xdata_val and also the dict, which is a little disturbing. (Yes, clearly not every place this occurs needs to be freed the same way.) Change-Id: Ic306d60b157e97c822a562bfdf21896e40db632a BUG: 1036102 Signed-off-by: Kaleb S. KEITHLEY <> Reviewed-on: Tested-by: Gluster Build System <> Reviewed-by: Vijay Bellur <>
Diffstat (limited to 'glusterfsd')
1 files changed, 5 insertions, 0 deletions
diff --git a/glusterfsd/src/glusterfsd-mgmt.c b/glusterfsd/src/glusterfsd-mgmt.c
index bbfc50593..0484779ce 100644
--- a/glusterfsd/src/glusterfsd-mgmt.c
+++ b/glusterfsd/src/glusterfsd-mgmt.c
@@ -1456,7 +1456,12 @@ glusterfs_volfile_fetch (glusterfs_ctx_t *ctx)
ret = mgmt_submit_request (&req, frame, ctx, &clnt_handshake_prog,
GF_HNDSK_GETSPEC, mgmt_getspec_cbk,
+ GF_FREE (req.xdata.xdata_val);
+ if (dict)
+ dict_unref (dict);
return ret;