diff options
| author | Sanju Rakonde <srakonde@redhat.com> | 2017-11-08 09:29:49 +0530 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2017-11-15 16:06:25 +0000 |
| commit | 3a0d9584e772191336854b140fadd3b124714012 (patch) | |
| tree | 1da11fd11bfb61bfb587fe1f67854d6801e13b43 /xlators/mgmt/glusterd/src/glusterd-handshake.c | |
| parent | a0b3d6534b5c4e84742ee9962a49a02009c637cd (diff) | |
glusterd: Fix coverity issues in handshake module
This patch fixes coveruty issues 737,738,172,188,728,198,235,35.
Change-Id: I632ddc5edbfb03bc81ce27f02886b8e262c1d946
BUG: 789278
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handshake.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handshake.c | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handshake.c b/xlators/mgmt/glusterd/src/glusterd-handshake.c index d4cd87676b0..56ddce88d5e 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handshake.c +++ b/xlators/mgmt/glusterd/src/glusterd-handshake.c @@ -455,7 +455,6 @@ gotvolinfo: strcat (dup_volid, ".tcp"); break; default: - ret = -1; break; } } @@ -652,7 +651,7 @@ glusterd_create_missed_snap (glusterd_missed_snap_info *missed_snapinfo, goto out; } strncpy (brickinfo->device_path, device, - sizeof(brickinfo->device_path)); + sizeof(brickinfo->device_path) - 1); /* Update the backend file-system type of snap brick in * snap volinfo. */ @@ -869,7 +868,6 @@ __server_getspec (rpcsvc_request_t *req) char *brick_name = NULL; char *volume = NULL; char *tmp = NULL; - int cookie = 0; rpc_transport_t *trans = NULL; gf_getspec_req args = {0,}; gf_getspec_rsp rsp = {0,}; @@ -999,8 +997,6 @@ fail: if (op_errno) rsp.op_errno = gf_errno_to_error (op_errno); - if (cookie) - rsp.op_errno = cookie; if (!rsp.spec) rsp.spec = strdup (""); @@ -1073,7 +1069,6 @@ __server_event_notify (rpcsvc_request_t *req) fail: rsp.op_ret = ret; - if (need_rsp) glusterd_submit_reply (req, &rsp, NULL, 0, NULL, (xdrproc_t)xdr_gf_event_notify_rsp); @@ -1820,8 +1815,9 @@ gd_validate_peer_op_version (xlator_t *this, glusterd_peerinfo_t *peerinfo, ret = 0; out: - gf_msg_debug (this->name , 0, "Peer %s %s", peerinfo->hostname, - ((ret < 0) ? "rejected" : "accepted")); + gf_msg_debug ((this ? this->name : "glusterd") , 0, "Peer %s %s", + peerinfo->hostname, ((ret < 0) ? "rejected" : "accepted") + ); return ret; } @@ -1888,7 +1884,7 @@ __glusterd_mgmt_hndsk_version_ack_cbk (struct rpc_req *req, struct iovec *iov, ret = default_notify (this, GF_EVENT_CHILD_UP, NULL); if (GD_MODE_ON == peerctx->args.mode) { - ret = glusterd_event_connected_inject (peerctx); + (void) glusterd_event_connected_inject (peerctx); peerctx->args.req = NULL; } else if (GD_MODE_SWITCH_ON == peerctx->args.mode) { peerctx->args.mode = GD_MODE_ON; @@ -2283,7 +2279,7 @@ __glusterd_peer_dump_version_cbk (struct rpc_req *req, struct iovec *iov, ret = default_notify (this, GF_EVENT_CHILD_UP, NULL); if (GD_MODE_ON == peerctx->args.mode) { - ret = glusterd_event_connected_inject (peerctx); + (void) glusterd_event_connected_inject (peerctx); peerctx->args.req = NULL; } else if (GD_MODE_SWITCH_ON == peerctx->args.mode) { peerctx->args.mode = GD_MODE_ON; |