From 37675f6b666bf9b4e07c52272a9f885fbc7cfb0d Mon Sep 17 00:00:00 2001 From: Erik Skultety Date: Thu, 5 May 2016 13:42:52 +0200 Subject: [PATCH] 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 --- src/admin/admin_remote.c | 2 +- src/libvirt-admin.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/admin/admin_remote.c b/src/admin/admin_remote.c index 40fcddb5fb..632aaa86fb 100644 --- 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 d7a66ff1a4..140fff37b5 100644 --- 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;