From 24d39aa142d64a831164e0fee100a17d440313f8 Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Mon, 27 Feb 2012 10:00:23 +0000 Subject: Always check result of _set_vg_name() --- tools/lvcreate.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) (limited to 'tools/lvcreate.c') 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 " -- cgit