diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2012-02-27 10:00:23 +0000 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2012-02-27 10:00:23 +0000 |
commit | 24d39aa142d64a831164e0fee100a17d440313f8 (patch) | |
tree | 4e1277b67fe3d8db7138c30f2748ae2da966c523 /tools/lvcreate.c | |
parent | 7e25b8f932495c6853282208cd71a6da8343454c (diff) | |
download | lvm2-24d39aa142d64a831164e0fee100a17d440313f8.tar.gz lvm2-24d39aa142d64a831164e0fee100a17d440313f8.tar.xz lvm2-24d39aa142d64a831164e0fee100a17d440313f8.zip |
Always check result of _set_vg_name()
Diffstat (limited to 'tools/lvcreate.c')
-rw-r--r-- | tools/lvcreate.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/tools/lvcreate.c b/tools/lvcreate.c index ebc1a0ec..cd4bcfd2 100644 --- a/tools/lvcreate.c +++ b/tools/lvcreate.c @@ -93,8 +93,9 @@ static int _lvcreate_name_params(struct lvcreate_params *lp, lp->origin = ptr + 1; } - if (!lp->vg_name) - _set_vg_name(lp, extract_vgname(cmd, NULL)); + if (!lp->vg_name && + !_set_vg_name(lp, extract_vgname(cmd, NULL))) + return_0; if (!lp->vg_name) { log_error("The origin name should include the " @@ -115,8 +116,9 @@ static int _lvcreate_name_params(struct lvcreate_params *lp, if (!_set_vg_name(lp, extract_vgname(cmd, lp->pool))) return_0; - if (!lp->vg_name) - _set_vg_name(lp, extract_vgname(cmd, NULL)); + if (!lp->vg_name && + !_set_vg_name(lp, extract_vgname(cmd, NULL))) + return_0; if (!lp->vg_name) { log_error("The pool name should include the " |