qemu: snapshot: Remove unnecessary 'do_transaction' logic in qemuDomainSnapshotCreate...

Now that we never get to the actual snapshot code if there's nothing to
do we can remove the variable and surrounding logic.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index b19ef5c79b85e4498146c33db27fd8a89a07d84b..f623eaa4220c1eca2114138d4d8768d5d2cf5e59 100644 (file)
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -15238,7 +15238,6 @@ qemuDomainSnapshotCreateDiskActive(virQEMUDriverPtr driver,
 {
     qemuDomainObjPrivatePtr priv = vm->privateData;
     VIR_AUTOPTR(virJSONValue) actions = NULL;
-    bool do_transaction = false;
     int rc;
     int ret = -1;
     size_t i;
@@ -15274,32 +15273,28 @@ qemuDomainSnapshotCreateDiskActive(virQEMUDriverPtr driver,
                                                      &diskdata[i],
                                                      actions, reuse) < 0)
             goto cleanup;
-
-        do_transaction = true;
     }
 
-    if (do_transaction) {
-        if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0)
-            goto cleanup;
-
-        rc = qemuMonitorTransaction(priv->mon, &actions);
+    if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0)
+        goto cleanup;
 
-        if (qemuDomainObjExitMonitor(driver, vm) < 0)
-            rc = -1;
+    rc = qemuMonitorTransaction(priv->mon, &actions);
 
-        for (i = 0; i < ndiskdata; i++) {
-            qemuDomainSnapshotDiskDataPtr dd = &diskdata[i];
+    if (qemuDomainObjExitMonitor(driver, vm) < 0)
+        rc = -1;
 
-            virDomainAuditDisk(vm, dd->disk->src, dd->src, "snapshot", rc >= 0);
+    for (i = 0; i < ndiskdata; i++) {
+        qemuDomainSnapshotDiskDataPtr dd = &diskdata[i];
 
-            if (rc == 0)
-                qemuDomainSnapshotUpdateDiskSources(dd);
-        }
+        virDomainAuditDisk(vm, dd->disk->src, dd->src, "snapshot", rc >= 0);
 
-        if (rc < 0)
-            goto cleanup;
+        if (rc == 0)
+            qemuDomainSnapshotUpdateDiskSources(dd);
     }
 
+    if (rc < 0)
+        goto cleanup;
+
     ret = 0;
 
  cleanup: