diff options
author | Alasdair Kergon <agk@redhat.com> | 2007-03-09 21:25:33 +0000 |
---|---|---|
committer | Alasdair Kergon <agk@redhat.com> | 2007-03-09 21:25:33 +0000 |
commit | b9b260119ff5d21238e6f9af7312da56aa8b2a88 (patch) | |
tree | 315ba5e6fd4c755883b69dc776f7f4bdc445f5e5 /tools/vgsplit.c | |
parent | 9397354a333b4b271ee270018985ac95ad22df02 (diff) | |
download | lvm2-b9b260119ff5d21238e6f9af7312da56aa8b2a88.tar.gz lvm2-b9b260119ff5d21238e6f9af7312da56aa8b2a88.tar.xz lvm2-b9b260119ff5d21238e6f9af7312da56aa8b2a88.zip |
Add a few missing pieces of vgname command line validation.
Diffstat (limited to 'tools/vgsplit.c')
-rw-r--r-- | tools/vgsplit.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/tools/vgsplit.c b/tools/vgsplit.c index 1695605b..09d58c2e 100644 --- a/tools/vgsplit.c +++ b/tools/vgsplit.c @@ -219,11 +219,17 @@ int vgsplit(struct cmd_context *cmd, int argc, char **argv) return EINVALID_CMD_LINE; } - vg_name_from = argv[0]; - vg_name_to = argv[1]; + vg_name_from = skip_dev_dir(cmd, argv[0], NULL); + vg_name_to = skip_dev_dir(cmd, argv[1], NULL); argc -= 2; argv += 2; + if (!validate_name(vg_name_from)) { + log_error("Volume group name \"%s\" is invalid", + vg_name_from); + return ECMD_FAILED; + } + if (!strcmp(vg_name_to, vg_name_from)) { log_error("Duplicate volume group name \"%s\"", vg_name_from); return ECMD_FAILED; |