From: Peter Krempa <[email protected]>

Move the flag check to the top level to allow programatic introspection
of supported flags.

Signed-off-by: Peter Krempa <[email protected]>
---
 src/qemu/qemu_driver.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 8027b5775e..e6ad16f31b 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -7417,9 +7417,6 @@ qemuDomainAttachDeviceLiveAndConfig(virDomainObj *vm,
     unsigned int parse_flags = VIR_DOMAIN_DEF_PARSE_INACTIVE |
                                VIR_DOMAIN_DEF_PARSE_ABI_UPDATE;

-    virCheckFlags(VIR_DOMAIN_AFFECT_LIVE |
-                  VIR_DOMAIN_AFFECT_CONFIG, -1);
-
     cfg = virQEMUDriverGetConfig(driver);

     /* The config and live post processing address auto-generation algorithms
@@ -7512,6 +7509,9 @@ qemuDomainAttachDeviceFlags(virDomainPtr dom,
     virDomainObj *vm = NULL;
     int ret = -1;

+    virCheckFlags(VIR_DOMAIN_AFFECT_LIVE |
+                  VIR_DOMAIN_AFFECT_CONFIG, -1);
+
     if (!(vm = qemuDomainObjFromDomain(dom)))
         goto cleanup;

@@ -7660,9 +7660,6 @@ qemuDomainDetachDeviceLiveAndConfig(virQEMUDriver *driver,
     unsigned int parse_flags = VIR_DOMAIN_DEF_PARSE_SKIP_VALIDATE;
     g_autoptr(virDomainDef) vmdef = NULL;

-    virCheckFlags(VIR_DOMAIN_AFFECT_LIVE |
-                  VIR_DOMAIN_AFFECT_CONFIG, -1);
-
     cfg = virQEMUDriverGetConfig(driver);

     if ((flags & VIR_DOMAIN_AFFECT_CONFIG) &&
@@ -7796,6 +7793,9 @@ qemuDomainDetachDeviceFlags(virDomainPtr dom,
     virDomainObj *vm = NULL;
     int ret = -1;

+    virCheckFlags(VIR_DOMAIN_AFFECT_LIVE |
+                  VIR_DOMAIN_AFFECT_CONFIG, -1);
+
     if (!(vm = qemuDomainObjFromDomain(dom)))
         goto cleanup;

-- 
2.54.0

Reply via email to