On Tue, 2018-08-21 at 11:01 +0200, Andrea Bolognani wrote:
On Mon, 2018-08-20 at 18:52 +0200, Michal Privoznik wrote: [...]
- if (def->deviceType == VIR_DOMAIN_CHR_DEVICE_TYPE_CHANNEL && - def->targetType == VIR_DOMAIN_CHR_CHANNEL_TARGET_TYPE_GUESTFWD) { - VIR_DEBUG("Ignoring device address for gustfwd channel"); - } else if (virDomainDeviceInfoParseXML(xmlopt, node, - &def->info, flags) < 0) { + if (virDomainDeviceInfoParseXML(xmlopt, node, &def->info, flags) < 0) goto error; - } -
I agree that fixing Bug 1610072 is more important than preventing Bug 1172526 from showing up again, but it would be great if we could make it so both are fixed...
How about parsing the info and then clearing out the address only if it's a guestfwd channel? The existing virDomainDeviceInfoClear() is a bit too thorough, but perhaps you can introduce a new virDomainDeviceInfoClearAddress() that only zeroes out the address and use that here.
Given the issues with the approach I proposed, let's just cut our losses and merge your original attempt instead. Sorry for wasting your time :( Reviewed-by: Andrea Bolognani <abologna@redhat.com> -- Andrea Bolognani / Red Hat / Virtualization