[PATCH 5/8] virsh: Resolve Coverity DEADCODE

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Coverity complains that on the first pass through the for loop that
'params' cannot be true, thus the ternary setting to "&" cannot be
done - which is I believe true - thus this really is a deadcode
situation.

It doesn't complain on the following comparison for "spice" - of course
because we've set params = true now. I believe that's correct since we're
putting together the line and the first "param" sets the "?" while the rest
use "&".

Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx>
---
 tools/virsh-domain.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
index f80741c..a040e27 100644
--- a/tools/virsh-domain.c
+++ b/tools/virsh-domain.c
@@ -9824,6 +9824,7 @@ cmdDomDisplay(vshControl *ctl, const vshCmd *cmd)
 
         /* TLS Port */
         if (tls_port) {
+            /* coverity[dead_error_condition] */
             virBufferAsprintf(&buf,
                               "%stls-port=%d",
                               params ? "&" : "?",
-- 
1.9.3

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]