diff options
| author | Kartik_Burmee <kburmee@redhat.com> | 2017-11-14 13:59:14 +0530 |
|---|---|---|
| committer | Xavier Hernandez <jahernan@redhat.com> | 2017-11-14 16:15:26 +0000 |
| commit | 1cee2719d36e1ca8cd646c9ce94679014c5a28a4 (patch) | |
| tree | fe8c357f53476077310dca240f415db6298ca594 /xlators/mgmt/glusterd/src/glusterd-handshake.c | |
| parent | 6e2ce37341e5d600d8fd5648b39eec0dbdbe45ad (diff) | |
glusterd: dead_code coverity fix
issue: Execution cannot reach this statement: "rsp.op_errno = gf_errno_to_...".
function:__server_event_notify
fix: removed the if statement and the corresponding actions. Also, the variable was not being used anywhere else, so removed its declaration as well
Change-Id: I85259e276c482cc9c98b1a829426bcec7412ce3f
BUG: 789278
Signed-off-by: Kartik_Burmee <kburmee@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handshake.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handshake.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handshake.c b/xlators/mgmt/glusterd/src/glusterd-handshake.c index a649e9f61be..d4cd87676b0 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handshake.c +++ b/xlators/mgmt/glusterd/src/glusterd-handshake.c @@ -1025,7 +1025,6 @@ int32_t __server_event_notify (rpcsvc_request_t *req) { int32_t ret = -1; - int32_t op_errno = 0; gf_event_notify_req args = {0,}; gf_event_notify_rsp rsp = {0,}; dict_t *dict = NULL; @@ -1074,8 +1073,6 @@ __server_event_notify (rpcsvc_request_t *req) fail: rsp.op_ret = ret; - if (op_errno) - rsp.op_errno = gf_errno_to_error (op_errno); if (need_rsp) glusterd_submit_reply (req, &rsp, NULL, 0, NULL, |