From: John Ferlan <jferlan@redhat.com>
Date: Wed, 23 Sep 2015 22:13:57 +0000 (-0400)
Subject: qemu: Resolve Coverity CHECKED_RETURN
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=ace8e2276e1ca507e25b93649f93176124061a72;p=people%2Fliuw%2Flibxenctrl-split%2Flibvirt.git

qemu: Resolve Coverity CHECKED_RETURN

Coverity complains that return from virHookCall is not checked in
one place in qemuProcessStop.  Since the comment notes that we cannot
stop the operation even it if fails, just added the ignore_value.
---

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index a8a43aca0..f2586a183 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -5273,9 +5273,9 @@ void qemuProcessStop(virQEMUDriverPtr driver,
         char *xml = qemuDomainDefFormatXML(driver, vm->def, 0);
 
         /* we can't stop the operation even if the script raised an error */
-        virHookCall(VIR_HOOK_DRIVER_QEMU, vm->def->name,
-                    VIR_HOOK_QEMU_OP_STOPPED, VIR_HOOK_SUBOP_END,
-                    NULL, xml, NULL);
+        ignore_value(virHookCall(VIR_HOOK_DRIVER_QEMU, vm->def->name,
+                                 VIR_HOOK_QEMU_OP_STOPPED, VIR_HOOK_SUBOP_END,
+                                 NULL, xml, NULL));
         VIR_FREE(xml);
     }