diff options
| author | Atin Mukherjee <amukherj@redhat.com> | 2017-07-26 12:46:42 +0530 |
|---|---|---|
| committer | Jeff Darcy <jeff@pl.atyp.us> | 2017-07-26 20:37:23 +0000 |
| commit | c136024613c697fec87aaff3a070862b92c57977 (patch) | |
| tree | fdebfc054a8a505f2fdc15651f86c90ad8f08b11 /cli/src/cli.h | |
| parent | 555990188ae7fabd4ca36c07ddaa92a39dccc813 (diff) | |
cli: error out incomplete CLI commands
cli_cmd_bricks_parse () & cli_cmd_volume_remove_brick_parse () were not
handling the the error cases where the command is incomplete with
missing brick details which could lead to glusterd crashes.
Credit : george.lian@nokia.com
Change-Id: Ia6303457a2aa279465aa75d4e1cfcc948893d5de
BUG: 1396327
Signed-off-by: Atin Mukherjee <amukherj@redhat.com>
Reviewed-on: https://review.gluster.org/17870
Smoke: Gluster Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Jeff Darcy <jeff@pl.atyp.us>
Diffstat (limited to 'cli/src/cli.h')
| -rw-r--r-- | cli/src/cli.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/cli/src/cli.h b/cli/src/cli.h index 1e92d675199..a1be2b62d06 100644 --- a/cli/src/cli.h +++ b/cli/src/cli.h @@ -278,7 +278,8 @@ cli_cmd_volume_old_tier_parse (const char **words, int wordcount, int32_t cli_cmd_volume_remove_brick_parse (const char **words, int wordcount, - dict_t **options, int *question); + dict_t **options, int *question, + int *brick_count); int32_t cli_cmd_volume_replace_brick_parse (const char **words, int wordcount, |