summaryrefslogtreecommitdiffstats
path: root/cli/src
diff options
context:
space:
mode:
authorRavishankar N <ravishankar@redhat.com>2016-06-16 10:21:53 +0530
committerAtin Mukherjee <amukherj@redhat.com>2016-06-20 04:41:29 -0700
commitd1a4e534ba9283da9e75033f4a3a309d16176b24 (patch)
tree7e131ee262a3707afc001b58ce60056424c458a1 /cli/src
parent362f90b5612c0f38894684d1d6f3bd66a31fe5b1 (diff)
cli: fix crash in arbiter keyword parsing
Backport of http://review.gluster.org/#/c/14738/ A negative case like `gluster volume create volname arbiter 3 /bricks{1..3}` must not crash. 'arbiter' keyword is valid only for (3 way) replica volumes. The .t that is added will crash and create a core *without* the fix when run but will still pass all TESTs. Since the regression framework fails the .t if it creates a core, we can consider it a valid test 'that fails without the fix'. Change-Id: Ie2d7ced66025ea3617d30f6f823b22401e6d2fde BUG: 1348055 Signed-off-by: Ravishankar N <ravishankar@redhat.com> (cherry picked from commit b5c492dfea2d2e2075aa88d7153fba57b06e739d) Reviewed-on: http://review.gluster.org/14764 Smoke: Gluster Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Diffstat (limited to 'cli/src')
-rw-r--r--cli/src/cli-cmd-parser.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/cli/src/cli-cmd-parser.c b/cli/src/cli-cmd-parser.c
index b062adfab0f..574cca1895b 100644
--- a/cli/src/cli-cmd-parser.c
+++ b/cli/src/cli-cmd-parser.c
@@ -628,6 +628,11 @@ cli_cmd_volume_create_parse (struct cli_state *state, const char **words,
goto out;
index += ret;
type = GF_CLUSTER_TYPE_DISPERSE;
+ } else if ((strcmp (w, "arbiter") == 0)) {
+ cli_err ("arbiter option must be preceded by replica "
+ "option.");
+ ret = -1;
+ goto out;
} else {
GF_ASSERT (!"opword mismatch");
ret = -1;