Re: [PATCH v2 01/12] graphics: don't parse listens if socket attribute is present

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

On Wed, May 11, 2016 at 5:08 PM, Pavel Hrdina <phrdina@xxxxxxxxxx> wrote:
> If socket attribute is present we start VNC that listens only on that
> unix socket.  This makes the parser behave the same way as we actually
> use the socket attribute.
>
> Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> ---
>  src/conf/domain_conf.c                                            | 8 ++++++++
>  .../genericxml2xmloutdata/generic-graphics-vnc-socket-listen.xml  | 4 +---
>  2 files changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> index 571b7bf..df2258a 100644
> --- a/src/conf/domain_conf.c
> +++ b/src/conf/domain_conf.c
> @@ -10721,11 +10721,18 @@ virDomainGraphicsListensParseXML(virDomainGraphicsDefPtr def,
>      xmlNodePtr save = ctxt->node;
>      virDomainGraphicsListenDefPtr address = NULL;
>      char *listenAddr = NULL;
> +    char *socketPath = NULL;
>      int nListens;
>      int ret = -1;
>
>      ctxt->node = node;
>
> +    if (def->type == VIR_DOMAIN_GRAPHICS_TYPE_VNC &&
> +        (socketPath = virXMLPropString(node, "socket"))) {
> +        ret = 0;
> +        goto error;
> +    }

Even if we keep the <listen> for compatibility purposes, perhaps there
is a log/warning to give that it is ignored.

Reviewed-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>



> +
>      /* parse the <listen> subelements for graphics types that support it */
>      nListens = virXPathNodeSet("./listen", ctxt, &listenNodes);
>      if (nListens < 0)
> @@ -10778,6 +10785,7 @@ virDomainGraphicsListensParseXML(virDomainGraphicsDefPtr def,
>   error:
>      VIR_FREE(listenNodes);
>      VIR_FREE(listenAddr);
> +    VIR_FREE(socketPath);
>      ctxt->node = save;
>      return ret;
>  }
> diff --git a/tests/genericxml2xmloutdata/generic-graphics-vnc-socket-listen.xml b/tests/genericxml2xmloutdata/generic-graphics-vnc-socket-listen.xml
> index a3b82fe..d8742c6 100644
> --- a/tests/genericxml2xmloutdata/generic-graphics-vnc-socket-listen.xml
> +++ b/tests/genericxml2xmloutdata/generic-graphics-vnc-socket-listen.xml
> @@ -19,9 +19,7 @@
>      <controller type='pci' index='0' model='pci-root'/>
>      <input type='mouse' bus='ps2'/>
>      <input type='keyboard' bus='ps2'/>
> -    <graphics type='vnc' socket='/tmp/QEMUGuest1-vnc.sock'>
> -      <listen type='address' address='127.0.0.1'/>
> -    </graphics>
> +    <graphics type='vnc' socket='/tmp/QEMUGuest1-vnc.sock'/>
>      <video>
>        <model type='cirrus' vram='16384' heads='1' primary='yes'/>
>      </video>
> --
> 2.8.2
>
> --
> libvir-list mailing list
> libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list



-- 
Marc-André Lureau

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]