domain_conf: graphics: move listens formatting to relevant graphics types

Only VNC, RDP and Spice graphics types are using listen elements so call
the function only where it is needed.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 93bbbf71679b3d53b792e927bae717987b5a3257..439d896514e1f6378617679849a256127b29f891 100644 (file)
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -26462,6 +26462,7 @@ virDomainGraphicsDefFormatAudio(virBuffer *buf,
 
 static int
 virDomainGraphicsDefFormatVNC(virBuffer *attrBuf,
+                              virBuffer *childBuf,
                               virDomainGraphicsDef *def,
                               unsigned int flags)
 {
@@ -26527,6 +26528,8 @@ virDomainGraphicsDefFormatVNC(virBuffer *attrBuf,
 
     virDomainGraphicsAuthDefFormatAttr(attrBuf, &def->data.vnc.auth, flags);
 
+    virDomainGraphicsDefFormatListnes(childBuf, def, flags);
+
     return 0;
 }
 
@@ -26547,6 +26550,7 @@ virDomainGraphicsDefFormatSDL(virBuffer *attrBuf,
 
 static void
 virDomainGraphicsDefFormatRDP(virBuffer *attrBuf,
+                              virBuffer *childBuf,
                               virDomainGraphicsDef *def,
                               unsigned int flags)
 {
@@ -26567,6 +26571,8 @@ virDomainGraphicsDefFormatRDP(virBuffer *attrBuf,
         virBufferAddLit(attrBuf, " multiUser='yes'");
 
     virDomainGraphicsListenDefFormatAddr(attrBuf, glisten, flags);
+
+    virDomainGraphicsDefFormatListnes(childBuf, def, flags);
 }
 
 static void
@@ -26581,6 +26587,7 @@ virDomainGraphicsDefFormatDesktop(virBuffer *attrBuf,
 
 static int
 virDomainGraphicsDefFormatSpice(virBuffer *attrBuf,
+                                virBuffer *childBuf,
                                 virDomainGraphicsDef *def,
                                 unsigned int flags)
 {
@@ -26637,6 +26644,8 @@ virDomainGraphicsDefFormatSpice(virBuffer *attrBuf,
 
     virDomainGraphicsAuthDefFormatAttr(attrBuf, &def->data.spice.auth, flags);
 
+    virDomainGraphicsDefFormatListnes(childBuf, def, flags);
+
     return 0;
 }
 
@@ -26685,7 +26694,7 @@ virDomainGraphicsDefFormat(virBuffer *buf,
 
     switch (def->type) {
     case VIR_DOMAIN_GRAPHICS_TYPE_VNC:
-        if (virDomainGraphicsDefFormatVNC(&attrBuf, def, flags) < 0)
+        if (virDomainGraphicsDefFormatVNC(&attrBuf, &childBuf, def, flags) < 0)
             return -1;
         break;
 
@@ -26694,7 +26703,7 @@ virDomainGraphicsDefFormat(virBuffer *buf,
         break;
 
     case VIR_DOMAIN_GRAPHICS_TYPE_RDP:
-        virDomainGraphicsDefFormatRDP(&attrBuf, def, flags);
+        virDomainGraphicsDefFormatRDP(&attrBuf, &childBuf, def, flags);
         break;
 
     case VIR_DOMAIN_GRAPHICS_TYPE_DESKTOP:
@@ -26702,7 +26711,7 @@ virDomainGraphicsDefFormat(virBuffer *buf,
         break;
 
     case VIR_DOMAIN_GRAPHICS_TYPE_SPICE:
-        if (virDomainGraphicsDefFormatSpice(&attrBuf, def, flags) < 0)
+        if (virDomainGraphicsDefFormatSpice(&attrBuf, &childBuf, def, flags) < 0)
             return -1;
         break;
 
@@ -26718,8 +26727,6 @@ virDomainGraphicsDefFormat(virBuffer *buf,
         break;
     }
 
-    virDomainGraphicsDefFormatListnes(&childBuf, def, flags);
-
     if (def->type == VIR_DOMAIN_GRAPHICS_TYPE_SPICE) {
         g_auto(virBuffer) spiceBuf = VIR_BUFFER_INITIALIZER;