[libvirt] [PATCH 3/5] vircommand: Remove unnecessary sa_assert

John Ferlan jferlan at redhat.com
Thu Feb 14 16:42:03 UTC 2013


Changes from commit '3178df9a' removed the need for the sa_assert(infd).
---
 src/util/vircommand.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/util/vircommand.c b/src/util/vircommand.c
index 5837fee..aa3bce0 100644
--- a/src/util/vircommand.c
+++ b/src/util/vircommand.c
@@ -1853,9 +1853,6 @@ virCommandProcessIO(virCommandPtr cmd)
                 fds[i].fd == cmd->inpipe) {
                 int done;
 
-                /* Coverity 5.3.0 can't see that we only get here if
-                 * infd is in the set because it was non-negative.  */
-                sa_assert(infd != -1);
                 done = write(cmd->inpipe, cmd->inbuf + inoff,
                              inlen - inoff);
                 if (done < 0) {
-- 
1.7.11.7




More information about the libvir-list mailing list