diff options
| author | Girjesh Rajoria <grajoria@redhat.com> | 2017-09-23 02:00:56 +0530 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2017-09-26 14:09:26 +0000 |
| commit | a2bbf75d04e8173c2156a001538564d141a26d4a (patch) | |
| tree | 142dd4b6e076b1b83ae50ab05c39bf92d5d4afbe | |
| parent | 467f1f3c3037efac0497d50510247e62605c3f3a (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.c | 2 |
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; } |