On Wed, Sep 10, 2014 at 3:52 PM, Zeeshan Ali (Khattak) <zeeshanak@xxxxxxxxx> wrote: > On Wed, Sep 10, 2014 at 3:35 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: >> Since commit 9cf9ca434, spice_channel_iterate() will report a >> SPICE_CHANNEL_ERROR_IO error to library users when >> SpiceChannel::has_error is set. In particular, when the server side >> closes its SPICE sockets because the VM is being shut down, an IO error >> will get reported. Prior to this change, a channel-closed event was >> reported on graceful VM shutdowns as there was >> a g_socket_condition_check() guarding the emission of the IO error >> signal. >> >> This commit readds the g_socket_condition_check() test, but only when >> SpiceChannel::has_error is set. >> >> This fixes https://bugs.freedesktop.org/show_bug.cgi?id=83692 >> --- > > Just wanted to say that I tested this patch and it fixes the bug above. Anyone up for review? It would be nice to get this in and (if possible) even released before GNOME's code freeze break on Wednesday. Won't be bad if release before 3.14 release in two weeks either. -- Regards, Zeeshan Ali (Khattak) ________________________________________ Befriend GNOME: http://www.gnome.org/friends/ _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel