[libvirt] [PATCH 1/2] conf: Fix crash in virDomainDefCompatibleDevice
John Ferlan
jferlan at redhat.com
Thu Mar 1 13:03:38 UTC 2018
Commit id 'edae027c' blindly assumed that the passed @oldDev
parameter would not be NULL when calling virDomainDeviceGetInfo;
however, commit id 'b6a264e8' passed NULL for AttachDevice
callers under the premise that there wouldn't be a device
to check/update against.
Signed-off-by: John Ferlan <jferlan at redhat.com>
---
src/conf/domain_conf.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index d96b012b9..fcafc8b2f 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -27417,9 +27417,12 @@ virDomainDefCompatibleDevice(virDomainDefPtr def,
{
virDomainCompatibleDeviceData data = {
.newInfo = virDomainDeviceGetInfo(dev),
- .oldInfo = virDomainDeviceGetInfo(oldDev),
+ .oldInfo = NULL,
};
+ if (oldDev)
+ data.oldInfo = virDomainDeviceGetInfo(oldDev);
+
if (!virDomainDefHasUSB(def) &&
def->os.type != VIR_DOMAIN_OSTYPE_EXE &&
virDomainDeviceIsUSB(dev)) {
--
2.13.6
More information about the libvir-list
mailing list