summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGirjesh Rajoria <grajoria@redhat.com>2017-09-23 02:00:56 +0530
committerAtin Mukherjee <amukherj@redhat.com>2017-09-26 14:09:26 +0000
commita2bbf75d04e8173c2156a001538564d141a26d4a (patch)
tree142dd4b6e076b1b83ae50ab05c39bf92d5d4afbe
parent467f1f3c3037efac0497d50510247e62605c3f3a (diff)
Coverity Issue Fix: IDENTICAL_BRANCHES
Issue: Event identical_branches: The same code is executed when the condition "ret" is true or false, because the code in the if-then branch and after the if statement is identical. Function: glusterd_print_gsync_status_by_vol Fix: removed if and goto statement. Change-Id: I966d793c9f3b65487acfb07083a4039caf593105 BUG: 789278 Signed-off-by: Girjesh Rajoria <grajoria@redhat.com>
-rw-r--r--xlators/mgmt/glusterd/src/glusterd-handler.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c
index a3ff49de173..25898dbd1fb 100644
--- a/xlators/mgmt/glusterd/src/glusterd-handler.c
+++ b/xlators/mgmt/glusterd/src/glusterd-handler.c
@@ -5111,8 +5111,6 @@ glusterd_print_gsync_status_by_vol (FILE *fp, glusterd_volinfo_t *volinfo)
/* Ignoring ret as above function always returns ret = 0 */
ret = glusterd_print_gsync_status (fp, gsync_rsp_dict);
- if (ret)
- goto out;
out:
return ret;
}