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

cli: fix possible memory leaks

Browse Source 
BUG: 955548
Change-Id: Iae410712e7e6d7a76cd537c77f1919e3b4cdf6bb
Signed-off-by: Bala.FA <barumuga@redhat.com>
Reviewed-on: http://review.gluster.org/6328
Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
This commit is contained in:
Bala.FA 2013-11-21 16:35:40 +05:30 committed by Anand Avati
parent 6157e4198c
commit d13bf457d4
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
xlators/mgmt/glusterd/src/glusterd-op-sm.c
View File

@ -3268,6 +3268,9 @@ glusterd_op_modify_op_ctx (glusterd_op_t op, void *ctx)
uuid = gf_strdup (uuid_str);
ret = dict_set_dynstr (op_ctx, key,
uuid);
if (ret != 0) {
GF_FREE (uuid);
}
}
}
}

4
xlators/mgmt/glusterd/src/glusterd-utils.c
View File

@ -4658,8 +4658,10 @@ glusterd_add_brick_to_dict (glusterd_volinfo_t *volinfo,
memset (key, 0, sizeof (key));
snprintf (key, sizeof (key), "%s.peerid", base_key);
ret = dict_set_dynstr (dict, key, peer_id_str);
if (ret)
if (ret) {
GF_FREE (peer_id_str);
goto out;
}
memset (key, 0, sizeof (key));
snprintf (key, sizeof (key), "%s.port", base_key);