diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2017-02-19 19:08:48 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2017-02-19 22:59:43 +0100 |
commit | 87c89ac2791e9b443db8c009db67610cbd97b4f2 (patch) | |
tree | 224ea596e372e9298916b9d8c73ffe59fcf1c621 | |
parent | 13944738d49ee420cb8b6b9eab4984b0a01b700d (diff) | |
download | lvm2-87c89ac2791e9b443db8c009db67610cbd97b4f2.tar.gz |
lvresize: fix regressiong on accepting args
Commit f45b68940667563e1a8d3bbd5d30864c04381bdf caused regression
of lvresize -m and --type parameter
After fix this sequence may work when we also fix syntax description:
lvcreate -l1 -m1 -n lv1 vg
lvextend --type mirror -m1 -l+1 vg/lv1
-rw-r--r-- | tools/lvresize.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/tools/lvresize.c b/tools/lvresize.c index 28fd0fda5..1d59239b9 100644 --- a/tools/lvresize.c +++ b/tools/lvresize.c @@ -21,7 +21,7 @@ static int _lvresize_params(struct cmd_context *cmd, int argc, char **argv, const char *cmd_name = command_name(cmd); const char *type_str = arg_str_value(cmd, type_ARG, NULL); - if (type_str && (lp->segtype = get_segtype_from_string(cmd, type_str))) + if (type_str && !(lp->segtype = get_segtype_from_string(cmd, type_str))) return_0; if (!strcmp(cmd_name, "lvreduce")) @@ -122,10 +122,13 @@ static int _lvresize_params(struct cmd_context *cmd, int argc, char **argv, return 0; } - if ((lp->mirrors = arg_count(cmd, mirrors_ARG)) && - (arg_sign_value(cmd, mirrors_ARG, SIGN_NONE) == SIGN_MINUS)) { - log_error("Mirrors argument may not be signed."); - return 0; + if (arg_is_set(cmd, mirrors_ARG)) { + if (arg_sign_value(cmd, mirrors_ARG, SIGN_NONE) != SIGN_NONE) { + log_error("Mirrors argument may not be signed."); + return 0; + } + if ((lp->mirrors = arg_uint_value(cmd, mirrors_ARG, 0))) + lp->mirrors++; } if ((lp->stripes = arg_uint_value(cmd, stripes_ARG, 0)) && |