admin: Remove flags checking from virAdmConnectOpen public API

Unlike the previous commit, we do actually support one client-side only flag
VIR_CONNECT_NO_ALIASES, so besides removing the check for flags this flag
has to be masked out before sending a message to the daemon, otherwise it
would trigger an error when checking flags on the daemon side.

Signed-off-by: Erik Skultety <eskultet@redhat.com>

diff --git a/src/admin/admin_remote.c b/src/admin/admin_remote.c
index 40fcddb5fb910af2f4e576808d261c58264a1ad5..632aaa86fbb52076b5ad4df2c143ab5516e68f86 100644 (file)
--- a/src/admin/admin_remote.c
+++ b/src/admin/admin_remote.c
@@ -162,7 +162,7 @@ remoteAdminConnectOpen(virAdmConnectPtr conn, unsigned int flags)
 
     virObjectLock(priv);
 
-    args.flags = flags;
+    args.flags = flags & ~VIR_CONNECT_NO_ALIASES;
 
     if (virNetClientRegisterAsyncIO(priv->client) < 0) {
         VIR_DEBUG("Failed to add event watch, disabling events and support for"

diff --git a/src/libvirt-admin.c b/src/libvirt-admin.c
index d7a66ff1a4d19a82bc7cd1ed312a771c7dea72ce..140fff37b5dd60ffed122aae4010ec7633f4dbbf 100644 (file)
--- a/src/libvirt-admin.c
+++ b/src/libvirt-admin.c
@@ -185,7 +185,8 @@ virAdmGetDefaultURI(virConfPtr conf)
 /**
  * virAdmConnectOpen:
  * @name: uri of the daemon to connect to, NULL for default
- * @flags: extra flags; not used yet, so callers should always pass 0
+ * @flags: bitwise-OR of virConnectFlags; so far the only supported flag is
+ *         VIR_CONNECT_NO_ALIASES
  *
  * Opens connection to admin interface of the daemon.
  *
@@ -204,7 +205,6 @@ virAdmConnectOpen(const char *name, unsigned int flags)
 
     VIR_DEBUG("flags=%x", flags);
     virResetLastError();
-    virCheckFlagsGoto(VIR_CONNECT_NO_ALIASES, error);
 
     if (!(conn = virAdmConnectNew()))
         goto error;