diff --git a/WHATS_NEW b/WHATS_NEW index 31270d006..5bf9ec486 100644 --- a/WHATS_NEW +++ b/WHATS_NEW @@ -1,6 +1,7 @@ Version 2.02.94 - -==================================== - Drop unused call to uname() during clvmd initia.lization. +==================================== + Always check result of _set_vg_name() in lvcreate. + Drop unused call to uname() during clvmd initialization. Test allocation result in sysfs filter creation. Limit sscanf parameters with buffer size in clvmd get_initial_state(). Use const lv pointer for lv_is_active...() functions. diff --git a/tools/lvcreate.c b/tools/lvcreate.c index ebc1a0ec0..cd4bcfd2d 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 "