On 04/30/2012 12:55 PM, Daniel P. Berrange wrote:
On Mon, Apr 30, 2012 at 12:42:37PM -0400, Stefan Berger wrote:> /libvirt/daemon/libvirtd.c:168: > leaked_handle: Handle variable "stdinfd" going out of scope leaks the handle. This is refering to the 3rd line here: switch (nextpid) { case 0: /* grandchild */ return statuspipe[1]; Neither the original 'cleanup:' label, nor any of the 3 you added will ever execute in the codepath that coverity is complaining about.
Hm, dropping the libvirtd.c changes... Stefan -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list