[libvirt PATCH 1/7] qemu: qemuDomainObjExitMonitor: do not warn on unused result

This wrapper for qemuDomainObjExitMonitorInternal was extended by my commit dc2fd51fd727bbb6de172e0ca4b7dd307bb99180 to check whether the domain is still alive, because we were observing crashes if the QEMU process died while some of our APIs were in the monitor and the thread processing the EOF event freed the domain definition. This bug was fixed by: commit 81f50cb92d16643bcd749e3ab5b404b8b7cec643 qemu: Avoid calling qemuProcessStop without a job but we kept checking for the return value since. Remove the G_GNUC_WARN_UNUSED_RESULT attribute since all of the calls that could set def->id to -1 are protected by qemuProcessBeginStopJob and cannot happen while we have a job in the monitor. Signed-off-by: Ján Tomko <jtomko(a)redhat.com&gt; --- src/qemu/qemu_domain.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/qemu/qemu_domain.h b/src/qemu/qemu_domain.h index 6728ab047e..596add616d 100644 --- a/src/qemu/qemu_domain.h +++ b/src/qemu/qemu_domain.h @@ -489,8 +489,7 @@ void qemuDomainObjEnterMonitor(virQEMUDriver *driver, ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); int qemuDomainObjExitMonitor(virQEMUDriver *driver, virDomainObj *obj) - ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) - G_GNUC_WARN_UNUSED_RESULT; + ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); int qemuDomainObjEnterMonitorAsync(virQEMUDriver *driver, virDomainObj *obj, qemuDomainAsyncJob asyncJob) -- 2.31.1