diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2016-02-25 20:40:28 +0100 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2016-02-25 23:30:24 +0100 |
| commit | abd9618dd8b77ca1974074ae76fa6a08f1a687d9 (patch) | |
| tree | 7dedd2b092c42b5a123856eb3c971bb307bfb3f8 /tools | |
| parent | a68e601886e1c9b78345b4c33ce366451e2fe184 (diff) | |
| download | lvm2-abd9618dd8b77ca1974074ae76fa6a08f1a687d9.tar.gz | |
lvconvert: fix vg parameter
Since we want to read env LVM_VG_NAME vg names,
we cannot just check LV names which do contain '/'.
So before the patch commands like:
> lvconvert --repair vg
Before:
Please provide a valid volume group name
After:
Path required for Logical Volume "vg".
Please provide a valid volume group name
> LVM_VG_NAME=vg lvconvert --repair vg
Before:
Please provide a valid volume group name
After:
Can't find LV vg in VG vg
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/lvconvert.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/tools/lvconvert.c b/tools/lvconvert.c index e7394c00d..3ec6d59ee 100644 --- a/tools/lvconvert.c +++ b/tools/lvconvert.c @@ -173,8 +173,7 @@ static int _lvconvert_name_params(struct lvconvert_params *lp, if (!validate_restricted_lvname_param(cmd, &lp->vg_name, &lp->lv_split_name)) return_0; - if (strchr(lp->lv_name_full, '/') && - (vg_name = extract_vgname(cmd, lp->lv_name_full)) && + if ((vg_name = extract_vgname(cmd, lp->lv_name_full)) && lp->vg_name && strcmp(vg_name, lp->vg_name)) { log_error("Please use a single volume group name " "(\"%s\" or \"%s\")", vg_name, lp->vg_name); |