Re: [PATCH spice-gtk v2 1/4] session: warn with the original URI

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

 



> 
> From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> 
> Before:
> (lt-spicy:18372): GSpice-WARNING **: 10:57:21.246: Double set of 'port' in
> URI 'spice://foo'
> 
> After:
> (lt-spicy:18654): GSpice-WARNING **: 10:57:21.246: Double set of 'port' in
> URI 'spice://foo:23?port=24'
> 
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> ---
>  src/spice-session.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/spice-session.c b/src/spice-session.c
> index 2aabf58..e6db424 100644
> --- a/src/spice-session.c
> +++ b/src/spice-session.c
> @@ -545,7 +545,7 @@ static int spice_parse_uri(SpiceSession *session, const
> char *original_uri)
>          }
>          if (target_key) {
>              if (*target_key) {
> -                g_warning("Double set of '%s' in URI '%s'", key, uri);
> +                g_warning("Double set of '%s' in URI '%s'", key,
> original_uri);
>                  goto fail;
>              }
>              *target_key = g_uri_unescape_string(value, NULL);
> @@ -553,7 +553,7 @@ static int spice_parse_uri(SpiceSession *session, const
> char *original_uri)
>      }
>  
>      if (port == NULL && tls_port == NULL) {
> -        g_warning("Missing port or tls-port in spice URI '%s'", uri);
> +        g_warning("Missing port or tls-port in spice URI '%s'",
> original_uri);
>          goto fail;
>      }
>  

Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]