[libvirt] [PATCH] daemon: Always advertise libvirtd service

Osier Yang jyang at redhat.com
Tue Oct 11 11:46:49 UTC 2011


This is a regression introduced by new RPC codes, previously
we advertise the service via ssh even if the daemon doesn't
listen on TLS port (TCP is not choosed). Now the service is
only advertised when it listens on TLS or TCP port. This breaks
upper layer apps which intends to discover the service, such
as virt-manager.
---
 daemon/libvirtd.c      |    6 ++++++
 src/rpc/virnetserver.c |   22 ++++++++++++++--------
 2 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c
index d1bc3dd..aa58121 100644
--- a/daemon/libvirtd.c
+++ b/daemon/libvirtd.c
@@ -531,6 +531,12 @@ static int daemonSetupNetworking(virNetServerPtr srv,
 
             virNetTLSContextFree(ctxt);
         }
+    } else {
+        if (virNetServerAddService(srv, NULL,
+                                   config->mdns_adv ?
+                                   "_libvirt._tcp" :
+                                   NULL) < 0)
+            goto error;
     }
 
 #if HAVE_SASL
diff --git a/src/rpc/virnetserver.c b/src/rpc/virnetserver.c
index f739743..3cf36bc 100644
--- a/src/rpc/virnetserver.c
+++ b/src/rpc/virnetserver.c
@@ -582,12 +582,16 @@ int virNetServerAddService(virNetServerPtr srv,
 {
     virNetServerLock(srv);
 
-    if (VIR_EXPAND_N(srv->services, srv->nservices, 1) < 0)
-        goto no_memory;
+    if (svc) {
+        if (VIR_EXPAND_N(srv->services, srv->nservices, 1) < 0)
+            goto no_memory;
+    }
 
 #if HAVE_AVAHI
     if (mdnsEntryName) {
-        int port = virNetServerServiceGetPort(svc);
+        int port = 0;
+        if (svc)
+            port = virNetServerServiceGetPort(svc);
 
         if (!virNetServerMDNSAddEntry(srv->mdnsGroup,
                                       mdnsEntryName,
@@ -596,12 +600,14 @@ int virNetServerAddService(virNetServerPtr srv,
     }
 #endif
 
-    srv->services[srv->nservices-1] = svc;
-    virNetServerServiceRef(svc);
+    if (svc) {
+        srv->services[srv->nservices-1] = svc;
+        virNetServerServiceRef(svc);
 
-    virNetServerServiceSetDispatcher(svc,
-                                     virNetServerDispatchNewClient,
-                                     srv);
+        virNetServerServiceSetDispatcher(svc,
+                                         virNetServerDispatchNewClient,
+                                         srv);
+    }
 
     virNetServerUnlock(srv);
     return 0;
-- 
1.7.6




More information about the libvir-list mailing list