Coverity notes that if the virConnectListAllDomains returns a negative
value then the loop at the cleanup label that ends on numDomains will
have issues.
Signed-off-by: John Ferlan <jferlan(a)redhat.com>
---
src/qemu/qemu_driver.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 373b4d7..5ceed92 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -1004,9 +1004,11 @@ qemuStateStop(void)
ret = -1;
cleanup:
- for (i = 0; i < numDomains; i++)
- virDomainFree(domains[i]);
- VIR_FREE(domains);
+ if (domains) {
+ for (i = 0; i < numDomains; i++)
+ virDomainFree(domains[i]);
+ VIR_FREE(domains);
+ }
VIR_FREE(flags);
virObjectUnref(conn);
virObjectUnref(cfg);
--
1.9.3