[libvirt] [PATCH 6/6] Fix coverity complain in commandtest.c

John Ferlan jferlan at redhat.com
Mon Jan 13 17:05:18 UTC 2014

On 01/13/2014 11:12 AM, Pavel Hrdina wrote:
> For a "newfd1" the coverity tool thinks that the fd is closed in
> a "virCommandPassFD", but with "flags == 0" it cannot be never
> closed.

I think you meant "cannot be close" the never is a double negative.

> The code itself is ok, but coverity tool thinks that there is
> "double_close" of the "newfd1" and to prevent showing this error
> we simply add a comment before the proper close.
> This has been found by coverity.
> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> ---
>  tests/commandtest.c | 1 +
>  1 file changed, 1 insertion(+)

When this was first changed/submitted:


and thus seen by my nightly Coverity run, I remember asking Dan about
this, but I think in the end the thought was it was a Coverity bug. I
had always meant to go back to it to see if I could put together a
reproducer to submit to Coverity as a bug report, but never quite got
around to it.

I suppose though, this is a viable "way around" the message, although it
may be a good idea to still revisit as a Coverity issue assuming a
reproducer program could be put together.

ACK with the grammar fix from above


More information about the libvir-list mailing list