9ae3f9
From 179213798496448316547506da65dbd9fd741dfa Mon Sep 17 00:00:00 2001
9ae3f9
From: Atin Mukherjee <amukherj@redhat.com>
9ae3f9
Date: Wed, 24 Apr 2019 22:02:51 +0530
9ae3f9
Subject: [PATCH 436/449] glusterd: coverity fixes
9ae3f9
9ae3f9
Addresses the following:
9ae3f9
9ae3f9
* CID 1124776:  Resource leaks  (RESOURCE_LEAK) - Variable "aa" going out
9ae3f9
of scope leaks the storage it points to in glusterd-volgen.c
9ae3f9
9ae3f9
* Bunch of CHECKED_RETURN defects in the callers of synctask_barrier_init
9ae3f9
9ae3f9
* CID 1400742:  Program hangs  (LOCK) - adding annotation to fix this
9ae3f9
false positive
9ae3f9
9ae3f9
> upstream patch link: https://review.gluster.org/#/c/glusterfs/+/22615
9ae3f9
> Updates: bz#789278
9ae3f9
> Change-Id: I02f16e7eeb8c5cf72f7d0b29d00df4f03b3718b3
9ae3f9
> Signed-off-by: Atin Mukherjee <amukherj@redhat.com>
9ae3f9
9ae3f9
BUG: 1787310
9ae3f9
Change-Id: I02f16e7eeb8c5cf72f7d0b29d00df4f03b3718b3
9ae3f9
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
9ae3f9
Reviewed-on: https://code.engineering.redhat.com/gerrit/202626
9ae3f9
Tested-by: RHGS Build Bot <nigelb@redhat.com>
9ae3f9
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
9ae3f9
---
9ae3f9
 xlators/mgmt/glusterd/src/glusterd-handler.c |  6 ++++++
9ae3f9
 xlators/mgmt/glusterd/src/glusterd-mgmt.c    | 24 +++++++++++++++++++-----
9ae3f9
 xlators/mgmt/glusterd/src/glusterd-syncop.c  | 22 ++++++++++++++++++----
9ae3f9
 xlators/mgmt/glusterd/src/glusterd-volgen.c  |  5 +++--
9ae3f9
 4 files changed, 46 insertions(+), 11 deletions(-)
9ae3f9
9ae3f9
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c
9ae3f9
index 1f31e72..b8799ab 100644
9ae3f9
--- a/xlators/mgmt/glusterd/src/glusterd-handler.c
9ae3f9
+++ b/xlators/mgmt/glusterd/src/glusterd-handler.c
9ae3f9
@@ -3458,6 +3458,12 @@ glusterd_friend_remove(uuid_t uuid, char *hostname)
9ae3f9
     ret = glusterd_peerinfo_cleanup(peerinfo);
9ae3f9
 out:
9ae3f9
     gf_msg_debug(THIS->name, 0, "returning %d", ret);
9ae3f9
+    /* We don't need to do a mutex unlock of peerinfo->delete_lock as the same
9ae3f9
+     * will be anyway destroyed within glusterd_peerinfo_cleanup, coverity
9ae3f9
+     * though cries about it
9ae3f9
+     */
9ae3f9
+    /* coverity[LOCK] */
9ae3f9
+
9ae3f9
     return ret;
9ae3f9
 }
9ae3f9
 
9ae3f9
diff --git a/xlators/mgmt/glusterd/src/glusterd-mgmt.c b/xlators/mgmt/glusterd/src/glusterd-mgmt.c
9ae3f9
index a4915f3..1e185d7 100644
9ae3f9
--- a/xlators/mgmt/glusterd/src/glusterd-mgmt.c
9ae3f9
+++ b/xlators/mgmt/glusterd/src/glusterd-mgmt.c
9ae3f9
@@ -757,7 +757,10 @@ glusterd_mgmt_v3_initiate_lockdown(glusterd_op_t op, dict_t *dict,
9ae3f9
 
9ae3f9
     /* Sending mgmt_v3 lock req to other nodes in the cluster */
9ae3f9
     gd_syncargs_init(&args, NULL);
9ae3f9
-    synctask_barrier_init((&args));
9ae3f9
+    ret = synctask_barrier_init((&args));
9ae3f9
+    if (ret)
9ae3f9
+        goto out;
9ae3f9
+
9ae3f9
     peer_cnt = 0;
9ae3f9
 
9ae3f9
     RCU_READ_LOCK;
9ae3f9
@@ -1108,7 +1111,10 @@ glusterd_mgmt_v3_pre_validate(glusterd_op_t op, dict_t *req_dict,
9ae3f9
 
9ae3f9
     /* Sending Pre Validation req to other nodes in the cluster */
9ae3f9
     gd_syncargs_init(&args, req_dict);
9ae3f9
-    synctask_barrier_init((&args));
9ae3f9
+    ret = synctask_barrier_init((&args));
9ae3f9
+    if (ret)
9ae3f9
+        goto out;
9ae3f9
+
9ae3f9
     peer_cnt = 0;
9ae3f9
 
9ae3f9
     RCU_READ_LOCK;
9ae3f9
@@ -1458,7 +1464,10 @@ glusterd_mgmt_v3_brick_op(glusterd_op_t op, dict_t *op_ctx, dict_t *req_dict,
9ae3f9
 
9ae3f9
     /* Sending brick op req to other nodes in the cluster */
9ae3f9
     gd_syncargs_init(&args, op_ctx);
9ae3f9
-    synctask_barrier_init((&args));
9ae3f9
+    ret = synctask_barrier_init((&args));
9ae3f9
+    if (ret)
9ae3f9
+        goto out;
9ae3f9
+
9ae3f9
     peer_cnt = 0;
9ae3f9
 
9ae3f9
     RCU_READ_LOCK;
9ae3f9
@@ -1722,7 +1731,9 @@ glusterd_mgmt_v3_commit(glusterd_op_t op, dict_t *op_ctx, dict_t *req_dict,
9ae3f9
 
9ae3f9
     /* Sending commit req to other nodes in the cluster */
9ae3f9
     gd_syncargs_init(&args, op_ctx);
9ae3f9
-    synctask_barrier_init((&args));
9ae3f9
+    ret = synctask_barrier_init((&args));
9ae3f9
+    if (ret)
9ae3f9
+        goto out;
9ae3f9
     peer_cnt = 0;
9ae3f9
 
9ae3f9
     RCU_READ_LOCK;
9ae3f9
@@ -1963,7 +1974,10 @@ glusterd_mgmt_v3_post_validate(glusterd_op_t op, int32_t op_ret, dict_t *dict,
9ae3f9
 
9ae3f9
     /* Sending Post Validation req to other nodes in the cluster */
9ae3f9
     gd_syncargs_init(&args, req_dict);
9ae3f9
-    synctask_barrier_init((&args));
9ae3f9
+    ret = synctask_barrier_init((&args));
9ae3f9
+    if (ret)
9ae3f9
+        goto out;
9ae3f9
+
9ae3f9
     peer_cnt = 0;
9ae3f9
 
9ae3f9
     RCU_READ_LOCK;
9ae3f9
diff --git a/xlators/mgmt/glusterd/src/glusterd-syncop.c b/xlators/mgmt/glusterd/src/glusterd-syncop.c
9ae3f9
index 9e47d14..c78983a 100644
9ae3f9
--- a/xlators/mgmt/glusterd/src/glusterd-syncop.c
9ae3f9
+++ b/xlators/mgmt/glusterd/src/glusterd-syncop.c
9ae3f9
@@ -1191,7 +1191,12 @@ gd_lock_op_phase(glusterd_conf_t *conf, glusterd_op_t op, dict_t *op_ctx,
9ae3f9
     struct syncargs args = {0};
9ae3f9
 
9ae3f9
     this = THIS;
9ae3f9
-    synctask_barrier_init((&args));
9ae3f9
+    GF_VALIDATE_OR_GOTO("glusterd", this, out);
9ae3f9
+
9ae3f9
+    ret = synctask_barrier_init((&args));
9ae3f9
+    if (ret)
9ae3f9
+        goto out;
9ae3f9
+
9ae3f9
     peer_cnt = 0;
9ae3f9
 
9ae3f9
     RCU_READ_LOCK;
9ae3f9
@@ -1321,7 +1326,10 @@ stage_done:
9ae3f9
     }
9ae3f9
 
9ae3f9
     gd_syncargs_init(&args, aggr_dict);
9ae3f9
-    synctask_barrier_init((&args));
9ae3f9
+    ret = synctask_barrier_init((&args));
9ae3f9
+    if (ret)
9ae3f9
+        goto out;
9ae3f9
+
9ae3f9
     peer_cnt = 0;
9ae3f9
 
9ae3f9
     RCU_READ_LOCK;
9ae3f9
@@ -1449,7 +1457,10 @@ commit_done:
9ae3f9
     }
9ae3f9
 
9ae3f9
     gd_syncargs_init(&args, op_ctx);
9ae3f9
-    synctask_barrier_init((&args));
9ae3f9
+    ret = synctask_barrier_init((&args));
9ae3f9
+    if (ret)
9ae3f9
+        goto out;
9ae3f9
+
9ae3f9
     peer_cnt = 0;
9ae3f9
     origin_glusterd = is_origin_glusterd(req_dict);
9ae3f9
 
9ae3f9
@@ -1541,7 +1552,10 @@ gd_unlock_op_phase(glusterd_conf_t *conf, glusterd_op_t op, int *op_ret,
9ae3f9
         goto out;
9ae3f9
     }
9ae3f9
 
9ae3f9
-    synctask_barrier_init((&args));
9ae3f9
+    ret = synctask_barrier_init((&args));
9ae3f9
+    if (ret)
9ae3f9
+        goto out;
9ae3f9
+
9ae3f9
     peer_cnt = 0;
9ae3f9
 
9ae3f9
     if (cluster_lock) {
9ae3f9
diff --git a/xlators/mgmt/glusterd/src/glusterd-volgen.c b/xlators/mgmt/glusterd/src/glusterd-volgen.c
9ae3f9
index 6852f8e..16346e7 100644
9ae3f9
--- a/xlators/mgmt/glusterd/src/glusterd-volgen.c
9ae3f9
+++ b/xlators/mgmt/glusterd/src/glusterd-volgen.c
9ae3f9
@@ -4808,9 +4808,10 @@ nfs_option_handler(volgen_graph_t *graph, struct volopt_map_entry *vme,
9ae3f9
 
9ae3f9
         if (ret != -1) {
9ae3f9
             ret = gf_canonicalize_path(vme->value);
9ae3f9
-            if (ret)
9ae3f9
+            if (ret) {
9ae3f9
+                GF_FREE(aa);
9ae3f9
                 return -1;
9ae3f9
-
9ae3f9
+            }
9ae3f9
             ret = xlator_set_option(xl, aa, ret, vme->value);
9ae3f9
             GF_FREE(aa);
9ae3f9
         }
9ae3f9
-- 
9ae3f9
1.8.3.1
9ae3f9