[libvirt] [PATCH] domain: Improve error output for virDomainListGetStats

Luyao Huang lhuang at redhat.com
Tue Nov 4 06:51:25 UTC 2014


When pass flags --domain and --list-* to cmdDomstats,
a unsupport error will output from qemuConnectGetAllDomainStats.

error: unsupported flags (0x1) in function qemuConnectGetAllDomainStats

>From manual of virsh: The approaches can't be combined.

Improve error to:

error: --domain and --list-* flags are mutually exclusive

Signed-off-by: Luyao Huang <lhuang at redhat.com>
---
 src/libvirt-domain.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/libvirt-domain.c b/src/libvirt-domain.c
index 7dc3146..6ae6dd2 100644
--- a/src/libvirt-domain.c
+++ b/src/libvirt-domain.c
@@ -11053,6 +11053,19 @@ virDomainListGetStats(virDomainPtr *doms,
         goto cleanup;
     }
 
+    if (flags & (VIR_CONNECT_LIST_DOMAINS_ACTIVE |
+                 VIR_CONNECT_LIST_DOMAINS_INACTIVE |
+                 VIR_CONNECT_LIST_DOMAINS_PERSISTENT |
+                 VIR_CONNECT_LIST_DOMAINS_TRANSIENT |
+                 VIR_CONNECT_LIST_DOMAINS_RUNNING |
+                 VIR_CONNECT_LIST_DOMAINS_SHUTOFF |
+                 VIR_CONNECT_LIST_DOMAINS_PAUSED |
+                 VIR_CONNECT_LIST_DOMAINS_OTHER)) {
+        virReportInvalidArg(flags, "%s",
+                    _("--domain and --list-* flags are mutually exclusive"));
+        goto cleanup;
+    }
+
     conn = doms[0]->conn;
     virCheckConnectReturn(conn, -1);
 
-- 
1.8.3.1




More information about the libvir-list mailing list