[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