From 11b1dd70e46b797f85af72b89eb6c3ebf739470c Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Wed, 27 Aug 2014 09:14:16 -0400 Subject: [PATCH] qemu_driver: Resolve Coverity CONSTANT_EXPRESSION_RESULT The call to virDomainSnapshotRedefinePrep() had a spurrious ! in front of it which caused Coverity to complan that the expression is always false. (cherry picked from commit 9d7254de430f7ef7a4c6a2a20718942369ce9d06) --- src/qemu/qemu_driver.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index c46226ad0d..9548542b9f 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -13527,8 +13527,8 @@ qemuDomainSnapshotCreateXML(virDomainPtr domain, } if (redefine) { - if (!virDomainSnapshotRedefinePrep(domain, vm, &def, &snap, - &update_current, flags) < 0) + if (virDomainSnapshotRedefinePrep(domain, vm, &def, &snap, + &update_current, flags) < 0) goto cleanup; } else { /* Easiest way to clone inactive portion of vm->def is via -- 2.47.3