[libvirt PATCH] Partially revert "qemu: fix missing error reports in capabilities probing"
Michal Prívozník
mprivozn at redhat.com
Fri Jul 17 08:03:49 UTC 2020
On 7/15/20 5:08 PM, Daniel P. Berrangé wrote:
> This partially reverts commit 5331c4804f4f419b9e75741084f926e52413d3a1.
>
> The original commit mistakenly thought virFileCacheLookup did not set
> an error. In fact the only case it doesn't set an error for is when
> the cache key is NULL. This in fact the fault of the caller for passing
> an invalid cache key, so doesn't need to be handled.
>
> This caller bug was fixed by checking for a NULL binary in the
> virQEMUCapsCacheLookupDefault method.
>
> Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> ---
> src/qemu/qemu_capabilities.c | 5 -----
> src/qemu/qemu_domain.c | 4 +---
> 2 files changed, 1 insertion(+), 8 deletions(-)
>
Reviewed-by: Michal Privoznik <mprivozn at redhat.com>
Michal
More information about the libvir-list
mailing list