diff options
author | Bhumika Goyal <bgoyal@redhat.com> | 2018-08-18 00:06:58 +0530 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-08-20 12:10:45 +0000 |
commit | 788cda4cd36574092bef1449ecda579163d06776 (patch) | |
tree | e1a2d6fe8fb4b81a76f7fb9c3eec843dbe17f734 /xlators/mgmt/glusterd/src/glusterd-utils.c | |
parent | 21199aaeb272da4b7857351bb2aa51f36c7e22cd (diff) |
glusterd: fix some coverity issues
Fixes CID: 1241481 1241482 1274079 1274118 1274121 1274131 1274198
1274214 1274220 1274224 1394663 1394641 382454 1382453 1382449 1288095
Link: https://scan6.coverity.com/reports.htm#v42388/p10714/fileInstanceId=84772667&defectInstanceId=25770661&mergedDefectId=744716
Change-Id: Idaf434186231c8b0fff4b27c57fa23636a89c8a7
updates: bz#789278
Signed-off-by: Bhumika Goyal <bgoyal@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-utils.c')
-rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-utils.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c index 7ede0245808..0f7fef65687 100644 --- a/xlators/mgmt/glusterd/src/glusterd-utils.c +++ b/xlators/mgmt/glusterd/src/glusterd-utils.c @@ -6639,6 +6639,8 @@ _local_gsyncd_start (dict_t *this, char *key, data_t *value, void *data) out: if (statefile) GF_FREE (statefile); + if (slave_url) + GF_FREE (slave_url); if (is_template_in_use) { op_ret = glusterd_create_status_file (volinfo->volname, slave, @@ -10270,6 +10272,8 @@ glusterd_volume_status_copy_to_op_ctx_dict (dict_t *aggr, dict_t *rsp_dict) } else { ret = dict_get_int32 (ctx_dict, "brick-index-max", &brick_index_max); + if (ret) + goto out; } rsp_ctx.count = node_count; @@ -13358,6 +13362,8 @@ out: "option %s does not exist", orig_key); *op_errstr = gf_strdup (err_str); } + if (def_val) + GF_FREE (def_val); gf_msg_debug (this->name, 0, "Returning %d", ret); return ret; } |