[libvirt] [PATCH 2/5] libvirt.c: Return error when remoteOnly is set but server is empty
Marcos Paulo de Souza
marcos.souza.org at gmail.com
Mon Jul 9 16:22:20 UTC 2018
Some drivers require a server in order to work, so this flag removes the
burden of esach driver to check for an server by doing it in
virConnectOpenInternal.
Signed-off-by: Marcos Paulo de Souza <marcos.souza.org at gmail.com>
---
src/libvirt.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/src/libvirt.c b/src/libvirt.c
index 52f4dd2808..9ae4e774eb 100644
--- a/src/libvirt.c
+++ b/src/libvirt.c
@@ -1005,6 +1005,11 @@ virConnectOpenInternal(const char *name,
continue;
}
+ if (virConnectDriverTab[i]->remoteOnly && ret->uri && !ret->uri->server) {
+ virReportError(VIR_ERR_INVALID_ARG, "%s", _("URI is missing the server part"));
+ goto failed;
+ }
+
/* Filter drivers based on declared URI schemes */
if (virConnectDriverTab[i]->uriSchemes) {
bool matchScheme = false;
--
2.17.1
More information about the libvir-list
mailing list