Re: [Spice-devel] [PATCH spice-gtk 2/2] channel: Abort migration in delayed unref

2016-04-30 Thread Victor Toso
Hi, On Tue, Apr 26, 2016 at 01:17:58PM +0200, Pavel Grunt wrote: > Hi, > > On Tue, 2016-04-26 at 12:05 +0200, Victor Toso wrote: > > Hi, > > > > On Fri, Apr 22, 2016 at 04:47:48PM +0200, Pavel Grunt wrote: > > > When channel is unref'ed during migration migrate_channel_event_cb > > > is called

Re: [Spice-devel] [PATCH spice-gtk 2/2] channel: Abort migration in delayed unref

2016-04-26 Thread Pavel Grunt
Hi, On Tue, 2016-04-26 at 12:05 +0200, Victor Toso wrote: > Hi, > > On Fri, Apr 22, 2016 at 04:47:48PM +0200, Pavel Grunt wrote: > > When channel is unref'ed during migration migrate_channel_event_cb > > is called causing a crash by coroutine yielding to nonexistent channel. > > > > As comment

Re: [Spice-devel] [PATCH spice-gtk 2/2] channel: Abort migration in delayed unref

2016-04-26 Thread Victor Toso
Hi, On Fri, Apr 22, 2016 at 04:47:48PM +0200, Pavel Grunt wrote: > When channel is unref'ed during migration migrate_channel_event_cb > is called causing a crash by coroutine yielding to nonexistent channel. > > As comment in spice_channel_coroutine says: > Co-routine exits now - the

[Spice-devel] [PATCH spice-gtk 2/2] channel: Abort migration in delayed unref

2016-04-22 Thread Pavel Grunt
When channel is unref'ed during migration migrate_channel_event_cb is called causing a crash by coroutine yielding to nonexistent channel. As comment in spice_channel_coroutine says: Co-routine exits now - the SpiceChannel object may no longer exist, so don't do anything else now unless you