Re: [libvirt] [PATCH 1/4] qemu: hotplug: Provide a string of a subsystem type instead of an int

On Tue, Mar 27, 2018 at 10:57:13 +0200, Erik Skultety wrote: > If one tries to detach a non-existent device, the error they get is: > "Unexpected hostdev type <int>". Let's use ToString conversion, since > the XML parser would have complained already if the type to be unplugged > was unknown to libvirt. > > Signed-off-by: Erik Skultety <eskultet(a)redhat.com&gt; > --- > src/qemu/qemu_hotplug.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c > index 49af4d4ff..6ec401e21 100644 > --- a/src/qemu/qemu_hotplug.c > +++ b/src/qemu/qemu_hotplug.c > @@ -5077,7 +5077,8 @@ int qemuDomainDetachHostDevice(virQEMUDriverPtr driver, > break; > default: > virReportError(VIR_ERR_INTERNAL_ERROR, > - _("unexpected hostdev type %d"), subsys->type); > + _("unexpected hostdev type '%s'"), > + virDomainHostdevSubsysTypeToString(subsys->type)); If the type is out of range of the enum this will return NULL. So you need to use NULLSTR() ACK with that tweak