[libvirt] [PATCH] virsh: correct return value error
Michal Privoznik
mprivozn at redhat.com
Wed Nov 30 09:20:52 UTC 2011
On 30.11.2011 09:25, ajia at redhat.com wrote:
> From: Alex Jia <ajia at redhat.com>
>
> The 'true' bool value should be 1, and false is 0.
>
> https://bugzilla.redhat.com/show_bug.cgi?id=758590
>
> Signed-off-by: Alex Jia <ajia at redhat.com>
> ---
> tools/virsh.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/tools/virsh.c b/tools/virsh.c
> index 01da8e5..618b0c1 100644
> --- a/tools/virsh.c
> +++ b/tools/virsh.c
> @@ -1745,7 +1745,7 @@ cmdDomblklist(vshControl *ctl, const vshCmd *cmd)
> VIR_FREE(source);
> }
>
> - ret = 0;
> + ret = true;
>
> cleanup:
> VIR_FREE(disks);
I'd change commit message:
Fix cmdDomblklist to return true on success instead of 0
But I can live with your's too.
ACK
Michal
More information about the libvir-list
mailing list