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@redhat.com> --- tests/commandtest.c | 1 + 1 file changed, 1 insertion(+)
When this was first changed/submitted: http://www.redhat.com/archives/libvir-list/2013-July/msg00831.html 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 John