On Thu, Sep 25, 2014 at 11:12:57AM +0200, Jiri Denemark wrote: > On Thu, Sep 25, 2014 at 10:47:55 +0200, Guido Günther wrote: > > Prompted by > > > > http://bugs.debian.org/761131 > > --- > > src/qemu/qemu_capabilities.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c > > index 49f5f75..dca00b2 100644 > > --- a/src/qemu/qemu_capabilities.c > > +++ b/src/qemu/qemu_capabilities.c > > @@ -3306,6 +3306,7 @@ virQEMUCapsInitQMP(virQEMUCapsPtr qemuCaps, > > virCommandAbort(cmd); > > virCommandFree(cmd); > > VIR_FREE(monarg); > > + ignore_value(unlink(monpath)); > > VIR_FREE(monpath); > > virObjectUnref(vm); > > virObjectUnref(xmlopt); > > ACK Pushed. I've added a check for non-null monpath although glibc seems to be happy with a NULL argument to unlink I couldn't find this behaviour specified in POSIX. Thanks, -- Guido -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list