From 9820756cd33467815af6d68b2ceef4e567646ec4 Mon Sep 17 00:00:00 2001 From: Nikolay Shirokovskiy Date: Thu, 31 Aug 2017 12:01:44 +0300 Subject: [PATCH] qemu: handle -1 for pid in qemuDomainGetMachineName We call qemuDomainGetMachineName on domain start. On first start (after daemon start) pid is 0 and virSystemdGetMachineNameByPID don't get called. But after domain shutting down pid became -1 so on next start virSystemdGetMachineNameByPID is called and returned an error. Error is ignored so it is not critical. But at least on my system (systemd-219 with extra patches) systemd-machined is crashed on this request. This behaviour is triggered by eaf2c9f89. Reviewed-by: Daniel P. Berrange --- src/qemu/qemu_domain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index cbee151f5d..b98ffffae2 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -9781,7 +9781,7 @@ qemuDomainGetMachineName(virDomainObjPtr vm) virQEMUDriverPtr driver = priv->driver; char *ret = NULL; - if (vm->pid) { + if (vm->pid > 0) { ret = virSystemdGetMachineNameByPID(vm->pid); if (!ret) virResetLastError(); -- 2.47.2