On Wed, Jun 14, 2017 at 21:02:19 +0530, Nitesh Konkar wrote:
> The advisory nodeset from numad is parsed irrespective of
> the string being empty or not. Avoid parsing an empty nodeset
> string.
>
> Signed-off-by: Nitesh Konkar <nitkon12@linux.vnet.ibm.com>
> ---
> src/qemu/qemu_process.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index 85b800d..1c49fa6 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -5456,7 +5456,7 @@ qemuProcessPrepareDomain(virConnectPtr conn,
> if (virDomainDefNeedsPlacementAdvice(vm->def)) {
> nodeset = virNumaGetAutoPlacementAdvice(virDomainDefGetVcpus(vm->def),
> virDomainDefGetMemoryTotal(vm->def));
> - if (!nodeset)
> + if (virStringIsEmpty(nodeset))
> goto cleanup;
>
> VIR_DEBUG("Nodeset returned from numad: %s", nodeset);
virNumaGetAutoPlacementAdvice returns NULL on error, any other value is
considered success. Thus this change could cause a jump to the cleanup
label and returning error from qemuProcessPrepareDomain without setting
any error.
NACK
Jirka
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list