]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commitdiff
use new macro helpers to check flag requirements
authorPavel Hrdina <phrdina@redhat.com>
Tue, 28 Apr 2015 17:24:07 +0000 (19:24 +0200)
committerPavel Hrdina <phrdina@redhat.com>
Mon, 4 May 2015 07:20:01 +0000 (09:20 +0200)
Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
src/libvirt-domain-snapshot.c
src/qemu/qemu_driver.c

index 9685c353e7b897287a648b2e1a8163849f973fe2..4625e038423f1aaefe3d8538f3c3669cbfe02800 100644 (file)
@@ -221,12 +221,9 @@ virDomainSnapshotCreateXML(virDomainPtr domain,
     virCheckNonNullArgGoto(xmlDesc, error);
     virCheckReadOnlyGoto(conn->flags, error);
 
-    if ((flags & VIR_DOMAIN_SNAPSHOT_CREATE_CURRENT) &&
-        !(flags & VIR_DOMAIN_SNAPSHOT_CREATE_REDEFINE)) {
-        virReportInvalidArg(flags, "%s",
-                            _("use of 'current' flag in requires 'redefine' flag"));
-        goto error;
-    }
+    VIR_REQUIRE_FLAG_GOTO(VIR_DOMAIN_SNAPSHOT_CREATE_CURRENT,
+                          VIR_DOMAIN_SNAPSHOT_CREATE_REDEFINE,
+                          error);
 
     VIR_EXCLUSIVE_FLAGS_GOTO(VIR_DOMAIN_SNAPSHOT_CREATE_REDEFINE,
                              VIR_DOMAIN_SNAPSHOT_CREATE_NO_METADATA,
index d6e7570ea4c3f98bdc4e08639862af4b8929d10e..d39d327cac89d285a39bb54104c9ae299b51b4df 100644 (file)
@@ -14943,12 +14943,9 @@ qemuDomainSnapshotCreateXML(virDomainPtr domain,
                   VIR_DOMAIN_SNAPSHOT_CREATE_ATOMIC |
                   VIR_DOMAIN_SNAPSHOT_CREATE_LIVE, NULL);
 
-    if ((flags & VIR_DOMAIN_SNAPSHOT_CREATE_QUIESCE) &&
-        !(flags & VIR_DOMAIN_SNAPSHOT_CREATE_DISK_ONLY)) {
-        virReportError(VIR_ERR_OPERATION_INVALID, "%s",
-                       _("quiesce requires disk-only"));
-        return NULL;
-    }
+    VIR_REQUIRE_FLAG_RET(VIR_DOMAIN_SNAPSHOT_CREATE_QUIESCE,
+                         VIR_DOMAIN_SNAPSHOT_CREATE_DISK_ONLY,
+                         NULL);
 
     if ((redefine && !(flags & VIR_DOMAIN_SNAPSHOT_CREATE_CURRENT)) ||
         (flags & VIR_DOMAIN_SNAPSHOT_CREATE_NO_METADATA))