Re: [spice-gtk] Don't report IO error on clean guest shutdown

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

 



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




[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]