[libvirt] [PATCH] Yet more coverity fixes
Stefan Berger
stefanb at linux.vnet.ibm.com
Mon Apr 30 17:06:24 UTC 2012
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
More information about the libvir-list
mailing list