
30 Mar
2022
30 Mar
'22
7:39 p.m.
On Wed, Mar 30, 2022 at 06:13:58PM +0100, Daniel P. Berrangé wrote:
On Wed, Mar 30, 2022 at 09:59:11AM -0700, Andrea Bolognani wrote:
+ if (virQEMUCapsGuestIsNative(hostarch, guestarch)) { + if ((ret = virFindFileInPath("/usr/libexec/qemu-kvm")) != NULL) { + return ret; + } + } + return ret; }
That overwrites the existing 'ret' value that we want to keep when qemu-kvm isn't present. Needs
if (!ret && virQEMUCapsGuestIsNative(hostarch, guestarch)) { ...
Does it? All previous attempts immediately return ret if it's not NULL, so if we get to this point we know that there's no existing value to preserve. -- Andrea Bolognani / Red Hat / Virtualization