diff options
| author | Michael Scherer <misc@redhat.com> | 2017-02-19 16:38:24 +0100 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2017-02-20 08:42:35 -0500 |
| commit | 37772e8e94400a60aedcd6713b1abedb1eb9ea7f (patch) | |
| tree | a2464f8950d6fd6a332bb0db76027450b65458a4 /xlators/mgmt/glusterd/src/glusterd-store.c | |
| parent | 73defab8be16b73241225bb1c2588a61e3e425d5 (diff) | |
Remove the useless goto
ret cannot be 0, since the error code for gf_store_save_value is
-1. And the label of the goto is just after the goto, so that's
deadcode.
Change-Id: I227bca41f4d0755891b8e6e0f4cb2ce004615a35
BUG: 1424809
Signed-off-by: Michael Scherer <misc@redhat.com>
Reviewed-on: https://review.gluster.org/16674
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
Tested-by: Nigel Babu <nigelb@redhat.com>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Prashanth Pai <ppai@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-store.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-store.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-store.c b/xlators/mgmt/glusterd/src/glusterd-store.c index 710a92c98c3..740354304c5 100644 --- a/xlators/mgmt/glusterd/src/glusterd-store.c +++ b/xlators/mgmt/glusterd/src/glusterd-store.c @@ -246,10 +246,6 @@ glusterd_store_volinfo_brick_fname_write (int vol_fd, glusterd_store_brickinfofname_set (brickinfo, brickfname, sizeof (brickfname)); ret = gf_store_save_value (vol_fd, key, brickfname); - if (ret) - goto out; - -out: return ret; } |