From a2bbf75d04e8173c2156a001538564d141a26d4a Mon Sep 17 00:00:00 2001 From: Girjesh Rajoria Date: Sat, 23 Sep 2017 02:00:56 +0530 Subject: 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 --- xlators/mgmt/glusterd/src/glusterd-handler.c | 2 -- 1 file changed, 2 deletions(-) (limited to 'xlators') 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; } -- cgit