|
 |
d1681e |
From 224af5bc6ea27a617f222cd83da871df85c6b7a1 Mon Sep 17 00:00:00 2001
|
|
|
d1681e |
From: Samikshan Bairagya <samikshan@gmail.com>
|
|
|
d1681e |
Date: Thu, 9 Nov 2017 15:15:37 +0530
|
|
|
d1681e |
Subject: [PATCH 182/201] glusterd: Fix coverity issues in glusterd-handler.c
|
|
|
d1681e |
|
|
|
d1681e |
Fixes get-state CLI related coverity issues 477, 511, 515, 523,
|
|
|
d1681e |
526 and 527 from the report at [1]
|
|
|
d1681e |
|
|
|
d1681e |
[1] https://download.gluster.org/pub/gluster/glusterfs/static-analysis/master/glusterfs-coverity/2017-10-30-9aa574a5/html/
|
|
|
d1681e |
|
|
|
d1681e |
>upstream mainline patch : https://review.gluster.org/#/c/18706/
|
|
|
d1681e |
|
|
|
d1681e |
Change-Id: Ieb6f64c9035b4d9338d9515de003d607b7a4e9bc
|
|
|
d1681e |
BUG: 1528733
|
|
|
d1681e |
Signed-off-by: Samikshan Bairagya <samikshan@gmail.com>
|
|
|
d1681e |
Reviewed-on: https://code.engineering.redhat.com/gerrit/132080
|
|
|
d1681e |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
d1681e |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
d1681e |
---
|
|
|
d1681e |
xlators/mgmt/glusterd/src/glusterd-handler.c | 34 ++++++++++++++++++++--------
|
|
|
d1681e |
1 file changed, 24 insertions(+), 10 deletions(-)
|
|
|
d1681e |
|
|
|
d1681e |
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c
|
|
|
d1681e |
index 81926a8..16a3773 100644
|
|
|
d1681e |
--- a/xlators/mgmt/glusterd/src/glusterd-handler.c
|
|
|
d1681e |
+++ b/xlators/mgmt/glusterd/src/glusterd-handler.c
|
|
|
d1681e |
@@ -5426,6 +5426,7 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
char *odir = NULL;
|
|
|
d1681e |
char *filename = NULL;
|
|
|
d1681e |
char *ofilepath = NULL;
|
|
|
d1681e |
+ char *tmp_str = NULL;
|
|
|
d1681e |
int count = 0;
|
|
|
d1681e |
int count_bkp = 0;
|
|
|
d1681e |
int odirlen = 0;
|
|
|
d1681e |
@@ -5435,6 +5436,7 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
uint64_t memtotal = 0;
|
|
|
d1681e |
uint64_t memfree = 0;
|
|
|
d1681e |
int start_index = 0;
|
|
|
d1681e |
+ char id_str[64] = {0,};
|
|
|
d1681e |
|
|
|
d1681e |
char *vol_type_str = NULL;
|
|
|
d1681e |
char *hot_tier_type_str = NULL;
|
|
|
d1681e |
@@ -5453,12 +5455,14 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
|
|
|
d1681e |
GF_VALIDATE_OR_GOTO (this->name, dict, out);
|
|
|
d1681e |
|
|
|
d1681e |
- ret = dict_get_str (dict, "odir", &odir;;
|
|
|
d1681e |
+ ret = dict_get_str (dict, "odir", &tmp_str);
|
|
|
d1681e |
if (ret) {
|
|
|
d1681e |
gf_asprintf (&odir, "%s", "/var/run/gluster/");
|
|
|
d1681e |
gf_msg (this->name, GF_LOG_INFO, 0,
|
|
|
d1681e |
GD_MSG_DICT_GET_FAILED,
|
|
|
d1681e |
"Default output directory: %s", odir);
|
|
|
d1681e |
+ } else {
|
|
|
d1681e |
+ gf_asprintf (&odir, "%s", tmp_str);
|
|
|
d1681e |
}
|
|
|
d1681e |
|
|
|
d1681e |
dp = sys_opendir (odir);
|
|
|
d1681e |
@@ -5481,7 +5485,7 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
goto out;
|
|
|
d1681e |
}
|
|
|
d1681e |
|
|
|
d1681e |
- ret = dict_get_str (dict, "filename", &filename);
|
|
|
d1681e |
+ ret = dict_get_str (dict, "filename", &tmp_str);
|
|
|
d1681e |
if (ret) {
|
|
|
d1681e |
now = time (NULL);
|
|
|
d1681e |
strftime (timestamp, sizeof (timestamp),
|
|
|
d1681e |
@@ -5491,6 +5495,8 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
gf_msg (this->name, GF_LOG_INFO, 0,
|
|
|
d1681e |
GD_MSG_DICT_GET_FAILED,
|
|
|
d1681e |
"Default filename: %s", filename);
|
|
|
d1681e |
+ } else {
|
|
|
d1681e |
+ gf_asprintf (&filename, "%s", tmp_str);
|
|
|
d1681e |
}
|
|
|
d1681e |
|
|
|
d1681e |
odirlen = strlen (odir);
|
|
|
d1681e |
@@ -5509,7 +5515,7 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
GF_FREE (odir);
|
|
|
d1681e |
GF_FREE (filename);
|
|
|
d1681e |
|
|
|
d1681e |
- ret = dict_set_dynstr (dict, "ofilepath", ofilepath);
|
|
|
d1681e |
+ ret = dict_set_str (dict, "ofilepath", ofilepath);
|
|
|
d1681e |
if (ret) {
|
|
|
d1681e |
gf_msg (this->name, GF_LOG_ERROR, 0,
|
|
|
d1681e |
GD_MSG_DICT_SET_FAILED, "Unable to set output path");
|
|
|
d1681e |
@@ -5564,7 +5570,9 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
|
|
|
d1681e |
fprintf (fp, "[Global]\n");
|
|
|
d1681e |
|
|
|
d1681e |
- fprintf (fp, "MYUUID: %s\n", gf_strdup (uuid_utoa (priv->uuid)));
|
|
|
d1681e |
+ uuid_utoa_r (priv->uuid, id_str);
|
|
|
d1681e |
+ fprintf (fp, "MYUUID: %s\n", id_str);
|
|
|
d1681e |
+
|
|
|
d1681e |
fprintf (fp, "op-version: %d\n", priv->op_version);
|
|
|
d1681e |
|
|
|
d1681e |
fprintf (fp, "\n[Global options]\n");
|
|
|
d1681e |
@@ -5656,8 +5664,10 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
}
|
|
|
d1681e |
|
|
|
d1681e |
fprintf (fp, "Volume%d.name: %s\n", ++count, volinfo->volname);
|
|
|
d1681e |
- fprintf (fp, "Volume%d.id: %s\n", count,
|
|
|
d1681e |
- gf_strdup (uuid_utoa (volinfo->volume_id)));
|
|
|
d1681e |
+
|
|
|
d1681e |
+ uuid_utoa_r (volinfo->volume_id, id_str);
|
|
|
d1681e |
+ fprintf (fp, "Volume%d.id: %s\n", count, id_str);
|
|
|
d1681e |
+
|
|
|
d1681e |
fprintf (fp, "Volume%d.type: %s\n", count, vol_type_str);
|
|
|
d1681e |
fprintf (fp, "Volume%d.transport_type: %s\n", count,
|
|
|
d1681e |
transport_type_str);
|
|
|
d1681e |
@@ -5777,8 +5787,11 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
fprintf (fp, "Volume%d.snapd_svc.inited: %s\n", count,
|
|
|
d1681e |
volinfo->snapd.svc.inited ? "True" : "False");
|
|
|
d1681e |
|
|
|
d1681e |
- fprintf (fp, "Volume%d.rebalance.id: %s\n", count,
|
|
|
d1681e |
- gf_strdup (uuid_utoa (volinfo->rebal.rebalance_id)));
|
|
|
d1681e |
+ uuid_utoa_r (volinfo->rebal.rebalance_id, id_str);
|
|
|
d1681e |
+ char *rebal_data = gf_uint64_2human_readable (
|
|
|
d1681e |
+ volinfo->rebal.rebalance_data);
|
|
|
d1681e |
+
|
|
|
d1681e |
+ fprintf (fp, "Volume%d.rebalance.id: %s\n", count, id_str);
|
|
|
d1681e |
fprintf (fp, "Volume%d.rebalance.status: %s\n", count,
|
|
|
d1681e |
rebal_status_str);
|
|
|
d1681e |
fprintf (fp, "Volume%d.rebalance.failures: %"PRIu64"\n", count,
|
|
|
d1681e |
@@ -5789,11 +5802,12 @@ glusterd_get_state (rpcsvc_request_t *req, dict_t *dict)
|
|
|
d1681e |
volinfo->rebal.lookedup_files);
|
|
|
d1681e |
fprintf (fp, "Volume%d.rebalance.files: %"PRIu64"\n", count,
|
|
|
d1681e |
volinfo->rebal.rebalance_files);
|
|
|
d1681e |
- fprintf (fp, "Volume%d.rebalance.data: %s\n", count,
|
|
|
d1681e |
- gf_uint64_2human_readable (volinfo->rebal.rebalance_data));
|
|
|
d1681e |
+ fprintf (fp, "Volume%d.rebalance.data: %s\n", count, rebal_data);
|
|
|
d1681e |
fprintf (fp, "Volume%d.time_left: %"PRIu64"\n", count,
|
|
|
d1681e |
volinfo->rebal.time_left);
|
|
|
d1681e |
|
|
|
d1681e |
+ GF_FREE (rebal_data);
|
|
|
d1681e |
+
|
|
|
d1681e |
if (volinfo->type == GF_CLUSTER_TYPE_TIER) {
|
|
|
d1681e |
ret = glusterd_volume_get_hot_tier_type_str (
|
|
|
d1681e |
volinfo, &hot_tier_type_str);
|
|
|
d1681e |
--
|
|
|
d1681e |
1.8.3.1
|
|
|
d1681e |
|