Re: [PATCH v1 20/37] Adapt to VIR_ALLOC and virAsprintf in src/qemu/*

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/04/2013 02:06 PM, Michal Privoznik wrote:
[...]
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index ba93233..7288ce7 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -10289,7 +10186,7 @@ virDomainDefPtr qemuParseCommandLine(virCapsPtr qemuCaps,
>  
>              if (!(disk->src || disk->nhosts > 0) ||
>                  !disk->dst)
> -                goto no_memory;
> +                goto error;
>  

I can't see how this can get true in the code, especially due to no
memory.  However, this is there already, so I'm more curious than
concerned.  But please check that any possible OOMError gets reported in
this path.

ACK,

Martin

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]