diff options
| author | Michael Adam <obnox@samba.org> | 2017-06-13 10:16:27 +0200 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2017-07-31 09:27:50 +0000 |
| commit | 96b179850d618c99d185ce7bfed5fe96e6cde919 (patch) | |
| tree | b9cf908c3a0c51bc788d1d1a2595b158dc46cbff /xlators/mgmt/glusterd/src/glusterd-server-quorum.c | |
| parent | e83ec8a65d0ef1a565ca66fa70d146b12ba1402a (diff) | |
glusterd: improve intuitivity of check in glusterd_get_quorum_cluster_counts
More intuitive to check for ret == 0 than !ret here...
Change-Id: I8177a0bc8f266331187f5f2eeefea8a25cfcb30a
Signed-off-by: Michael Adam <obnox@samba.org>
BUG: 1476410
Reviewed-on: https://review.gluster.org/17912
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Tested-by: Atin Mukherjee <amukherj@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-server-quorum.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-server-quorum.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-server-quorum.c b/xlators/mgmt/glusterd/src/glusterd-server-quorum.c index a4637f84368..37dec3d2d6b 100644 --- a/xlators/mgmt/glusterd/src/glusterd-server-quorum.c +++ b/xlators/mgmt/glusterd/src/glusterd-server-quorum.c @@ -235,7 +235,7 @@ glusterd_get_quorum_cluster_counts (xlator_t *this, int *active_count, ret = dict_get_str (conf->opts, GLUSTERD_QUORUM_RATIO_KEY, &val); if (ret == 0) { ret = gf_string2percent (val, &quorum_percentage); - if (!ret) + if (ret == 0) ratio = _gf_true; } if (ratio) |