From 11a9f9f0dcca58446f306ef2060a345348ed91c1 Mon Sep 17 00:00:00 2001 From: Vijay Bellur Date: Mon, 13 Aug 2018 14:52:09 -0700 Subject: mgmt/glusterd: Fix resource leak & unused value issues in glusterd-volume-ops.c Addresses CID: 1274132, 1325534 Change-Id: I176612ef5baf5618d543838a5f32db7dcd7002c3 updates: bz#789278 Signed-off-by: Vijay Bellur --- xlators/mgmt/glusterd/src/glusterd-volume-ops.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/xlators/mgmt/glusterd/src/glusterd-volume-ops.c b/xlators/mgmt/glusterd/src/glusterd-volume-ops.c index a02b0cb7571..030006b5459 100644 --- a/xlators/mgmt/glusterd/src/glusterd-volume-ops.c +++ b/xlators/mgmt/glusterd/src/glusterd-volume-ops.c @@ -197,6 +197,7 @@ glusterd_check_brick_order(dict_t *dict, char *err_str) gf_msg (this->name, GF_LOG_ERROR, ENOMEM, GD_MSG_NO_MEMORY, "failed to allocate " "memory"); + freeaddrinfo (ai_info); goto out; } ai_list_tmp1->info = ai_info; @@ -2290,6 +2291,8 @@ glusterd_op_create_volume (dict_t *dict, char **op_errstr) "replica count for volume %s", volname); goto out; } + + /* coverity[unused_value] arbiter count is optional */ ret = dict_get_int32 (dict, "arbiter-count", &volinfo->arbiter_count); } else if (GF_CLUSTER_TYPE_STRIPE == volinfo->type) { @@ -2335,6 +2338,8 @@ glusterd_op_create_volume (dict_t *dict, char **op_errstr) "replica count for volume %s", volname); goto out; } + + /* coverity[unused_value] arbiter count is optional */ ret = dict_get_int32 (dict, "arbiter-count", &volinfo->arbiter_count); } else if (GF_CLUSTER_TYPE_DISPERSE == volinfo->type) { -- cgit