[libvirt] [PATCH] Really fix XML formatting flags in SaveImageUpdateDef

Jiri Denemark jdenemar at redhat.com
Fri Feb 27 11:37:25 UTC 2015


On Fri, Feb 27, 2015 at 12:07:40 +0100, Ján Tomko wrote:
> Commit cf2d4c6 used a logical or instead of bitwise or,
> effectively passing 1, that is VIR_DOMAIN_XML_INACTIVE.
> Yep, that's right.
> 
> This was caught by a warning when building with clang.
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1183869
> ---
>  src/qemu/qemu_driver.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index e9aa0b5..e282464 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -5734,7 +5734,7 @@ qemuDomainSaveImageUpdateDef(virQEMUDriverPtr driver,
>  
>      if (!(newdef_migr = qemuDomainDefCopy(driver,
>                                            newdef,
> -                                          QEMU_DOMAIN_FORMAT_LIVE_FLAGS ||
> +                                          QEMU_DOMAIN_FORMAT_LIVE_FLAGS |
>                                            VIR_DOMAIN_XML_MIGRATABLE)))
>          goto cleanup;
>  

ACK

Jirka




More information about the libvir-list mailing list