[libvirt] [PATCH 3/4] domain_conf: Resolve Coverity CHECKED_RETURN

John Ferlan jferlan at redhat.com
Mon Dec 1 16:33:56 UTC 2014


Commit id '0d36a5d05' modified the code slightly, but removed the
return value check thus causing Coverity to complain that this call
was the only one where the return value wasn't checked. Since nothing
was done previously if there was a failure, just use ignore_value here
to pacify Coverity

Signed-off-by: John Ferlan <jferlan at redhat.com>
---
 src/conf/domain_conf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index b0caa76..fe9b986 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -19237,7 +19237,7 @@ virDomainDefFormatInternal(virDomainDefPtr def,
         virDomainResourceDefFormat(buf, def->resource);
 
     if (def->sysinfo)
-        virSysinfoFormat(buf, def->sysinfo);
+        ignore_value(virSysinfoFormat(buf, def->sysinfo));
 
     if (def->os.bootloader) {
         virBufferEscapeString(buf, "<bootloader>%s</bootloader>\n",
-- 
1.9.3




More information about the libvir-list mailing list