[lvm-devel] [PATCH] Do not try to unlock VG which is not locked.

[Petr Rockai]

Hi,

Milan Broz <mbroz at redhat.com> writes:
> # vgchange -a y x --ignorelockingfailure
>   Volume group "x" not found
>   Internal error: Attempt to unlock unlocked VG x
>
> Signed-off-by: Milan Broz <mbroz at redhat.com>
> ---
>  tools/toollib.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/tools/toollib.c b/tools/toollib.c
> index 39a6d69..33daf30 100644
> --- a/tools/toollib.c
> +++ b/tools/toollib.c
> @@ -452,8 +452,7 @@ static int _process_one_vg(struct cmd_context *cmd, const char *vg_name,
>  		ret_max = ret;
>  
>  out:
> -	if ((vg_read_error(vg) == FAILED_ALLOCATION)||
> -	    (vg_read_error(vg) == FAILED_LOCKING))
> +	if (vg_read_error(vg))
>  		vg_release(vg);
>  	else
>  		unlock_and_release_vg(cmd, vg, vg_name);
looks good to me. The error path in vg_read (metadata.c) makes sure that
the VG is unlocked if any errors have happened.

=> ACK.

Yours,
    Petr.