[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