Re: [PATCH spice-gtk] channel: swap channel GSocketConnection on migration

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

 



ACK

Christophe

On Fri, Nov 22, 2013 at 04:05:34PM +0100, Marc-André Lureau wrote:
> When migration completes, unrefing the new connection leads to original
> GSocket pending refs, and thus the sockets stay in CLOSE_WAIT.
> 
> This is a regression from 8029bd0 where GSocketConnection is kept around
> to satisfy old glib.
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1024501
> 
> This will also probably fix:
> https://bugzilla.redhat.com/show_bug.cgi?id=952375
> ---
>  gtk/spice-channel.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/gtk/spice-channel.c b/gtk/spice-channel.c
> index eda31d2..6ddbe50 100644
> --- a/gtk/spice-channel.c
> +++ b/gtk/spice-channel.c
> @@ -2733,6 +2733,7 @@ void spice_channel_swap(SpiceChannel *channel, SpiceChannel *swap, gboolean swap
>  
>      /* TODO: split channel in 2 objects: a controller and a swappable
>         state object */
> +    SWAP(conn);
>      SWAP(sock);
>      SWAP(ctx);
>      SWAP(ssl);
> -- 
> 1.8.4.2
> 

Attachment: pgpmioVyFdVXf.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://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]