Coverity complained that 4 out of 5 callers to virJSONValueObjectGetBoolean
checked for errors. But we documented that we don't care in this case.
* src/qemu/qemu_monitor_json.c (qemuMonitorJSONGetBlockInfo): Use
ignore_value.
---
Pushing under the trivial rule.
src/qemu/qemu_monitor_json.c | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index c4f8360..3d383c8 100644
--- a/src/qemu/qemu_monitor_json.c
+++ b/src/qemu/qemu_monitor_json.c
@@ -40,6 +40,7 @@
#include "datatypes.h"
#include "virterror_internal.h"
#include "json.h"
+#include "ignore-value.h"
#define VIR_FROM_THIS VIR_FROM_QEMU
@@ -1418,7 +1419,8 @@ int qemuMonitorJSONGetBlockInfo(qemuMonitorPtr mon,
/* Don't check for success here, because 'tray-open' is presented iff
* medium is ejected.
*/
- virJSONValueObjectGetBoolean(dev, "tray-open", &info->tray_open);
+ ignore_value(virJSONValueObjectGetBoolean(dev, "tray-open",
+ &info->tray_open));
break;
}
--
1.7.4.4
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list