qemu: bulk stats: Fix logic in monitor handling
authorFrancesco Romani <fromani@redhat.com>
Thu, 11 Dec 2014 07:44:09 +0000 (08:44 +0100)
committerMartin Kletzander <mkletzan@redhat.com>
Thu, 11 Dec 2014 13:11:06 +0000 (14:11 +0100)
commitdfbdea7ea8fa36d9f27942c5b2882acfd86a3c3b
tree2652f8b4f0a0eaf03f8be3c2836e3de08b0bad37
parent5d8bee6d57cddf462912ad2fc544c8a57b1c2841
qemu: bulk stats: Fix logic in monitor handling

A logic bug in qemuConnectGetAllDomainStats makes the code mark the
monitor as available when qemuDomainObjBeginJob fails, instead of when
it succeeds, as the correct flow requires.

This patch fixes the check and updates the code documentation
accordingly.

Broken by commit 57023c0a3af4af1c547189c1f6712ed5edeb0c0b.

Signed-off-by: Francesco Romani <fromani@redhat.com>
(cherry picked from commit cb104ef734dfea12cb8826dba7e2c98912c4b7e1)
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
src/qemu/qemu_driver.c