<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 25, 2016 at 4:06 PM, Pavel Hrdina <span dir="ltr"><<a href="mailto:phrdina@redhat.com" target="_blank">phrdina@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, May 25, 2016 at 03:49:04AM -0500, Shivaprasad G Bhat wrote:<br>
> Signed-off-by: Shivaprasad G Bhat <<a href="mailto:sbhat@linux.vnet.ibm.com">sbhat@linux.vnet.ibm.com</a>><br>
> ---<br>
>  src/qemu/qemu_driver.c |    4 ++--<br>
>  1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c<br>
> index 249393a..3f26f1f 100644<br>
> --- a/src/qemu/qemu_driver.c<br>
> +++ b/src/qemu/qemu_driver.c<br>
> @@ -8237,7 +8237,7 @@ qemuDomainAttachDeviceFlags(virDomainPtr dom,<br>
>      if (priv->qemuCaps)<br>
>          qemuCaps = virObjectRef(priv->qemuCaps);<br>
>      else if (!(qemuCaps = virQEMUCapsCacheLookup(driver->qemuCapsCache, vm->def->emulator)))<br>
> -        goto cleanup;<br>
> +        goto endjob;<br>
><br>
>      if (flags & VIR_DOMAIN_AFFECT_CONFIG) {<br>
>          /* Make a copy for updated domain. */<br>
> @@ -8490,7 +8490,7 @@ qemuDomainDetachDeviceFlags(virDomainPtr dom,<br>
>      if (priv->qemuCaps)<br>
>          qemuCaps = virObjectRef(priv->qemuCaps);<br>
>      else if (!(qemuCaps = virQEMUCapsCacheLookup(driver->qemuCapsCache, vm->def->emulator)))<br>
> -        goto cleanup;<br>
> +        goto endjob;<br>
><br>
>      if (flags & VIR_DOMAIN_AFFECT_CONFIG) {<br>
>          /* Make a copy for updated domain. */<br>
<br>
</div></div>The same bug is also on line 8369, could you fix that too and send a v2?<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Oh I see.. I scanned the whole file from AttachDeviceConfig() for this bug if exists anywhere else. Not sure</div><div>how I missed UpdateDeviceConfig() though.  :) Just sent the V2.</div><div><br></div><div>Thanks,</div><div>Shivaprasad</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
Pavel<br>
</font></span></blockquote></div><br></div></div>