From: John Ferlan Date: Wed, 27 Aug 2014 19:59:08 +0000 (-0400) Subject: qemu_monitor: Resolve Coverity NESTING_INDENT_MISMATCH X-Git-Tag: v1.2.8-rc2~11 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ee8b6245e96139f4a18c3ab34880594470a6cb71;p=thirdparty%2Flibvirt.git qemu_monitor: Resolve Coverity NESTING_INDENT_MISMATCH The PROBE macro can expand to more than one line/statement - put curly braces around the if statement to be safe --- diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c index d5ba08de97..5b2952aa44 100644 --- a/src/qemu/qemu_monitor.c +++ b/src/qemu/qemu_monitor.c @@ -506,10 +506,11 @@ qemuMonitorIOWrite(qemuMonitorPtr mon) mon->msg->txLength - mon->msg->txOffset, done, errno); - if (mon->msg->txFD != -1) + if (mon->msg->txFD != -1) { PROBE(QEMU_MONITOR_IO_SEND_FD, "mon=%p fd=%d ret=%d errno=%d", mon, mon->msg->txFD, done, errno); + } if (done < 0) { if (errno == EAGAIN)