Hi, On Thu, Jul 28, 2016 at 06:55:11PM +0200, Pavel Grunt wrote: > Create it on SPICE_PORT_EVENT_OPENED I would make reference to the g_cancellable_reset() documentation where it states one should not re-use a GCancellable if it was cancelled. With this change, we might not need to create it on _init? Cheers, > > Reusing the cancellable leads to a crash: > #0 0x00007ffff1662940 in g_task_return_error () at /lib64/libgio-2.0.so.0 > #1 0x00007ffff1662b60 in g_task_return_new_error () at /lib64/libgio-2.0.so.0 > #2 0x00007ffff57916e0 in read_cancelled (cancellable=<optimized out>, user_data=<optimized out>) at vmcstream.c:182 > #3 0x00007ffff1391555 in g_closure_invoke () at /lib64/libgobject-2.0.so.0 > #4 0x00007ffff13a4062 in signal_emit_unlocked_R () at /lib64/libgobject-2.0.so.0 > #5 0x00007ffff13acf5f in g_signal_emit_valist () at /lib64/libgobject-2.0.so.0 > #6 0x00007ffff13ad33f in g_signal_emit () at /lib64/libgobject-2.0.so.0 > #7 0x00007ffff16100d8 in g_cancellable_cancel () at /lib64/libgio-2.0.so.0 > #8 0x00007ffff5775c01 in port_event (self=0xc7af60, event=1) at channel-webdav.c:512 > #9 0x00007ffff1391555 in g_closure_invoke () at /lib64/libgobject-2.0.so.0 > #10 0x00007ffff13a445d in signal_emit_unlocked_R () at /lib64/libgobject-2.0.so.0 > #11 0x00007ffff13acf5f in g_signal_emit_valist () at /lib64/libgobject-2.0.so.0 > #12 0x00007ffff577479a in emit_main_context (opaque=0x7fffa5fff8d0) at gio-coroutine.c:200 > #13 0x00007ffff10b7847 in g_idle_dispatch () at /lib64/libglib-2.0.so.0 > #14 0x00007ffff10bade2 in g_main_context_dispatch () at /lib64/libglib-2.0.so.0 > #15 0x00007ffff10bb160 in g_main_context_iterate.isra () at /lib64/libglib-2.0.so.0 > #16 0x00007ffff10bb20c in g_main_context_iteration () at /lib64/libglib-2.0.so.0 > #17 0x00007ffff1677d2d in g_application_run () at /lib64/libgio-2.0.so.0 > #18 0x000000000041031a in main (argc=2, argv=0x7fffffffd928) at remote-viewer-main.c:42 > > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=97113 > --- > src/channel-webdav.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/channel-webdav.c b/src/channel-webdav.c > index 307a867..0a2ed72 100644 > --- a/src/channel-webdav.c > +++ b/src/channel-webdav.c > @@ -506,7 +506,8 @@ static void port_event(SpiceWebdavChannel *self, gint event) > > CHANNEL_DEBUG(self, "port event:%d", event); > if (event == SPICE_PORT_EVENT_OPENED) { > - g_cancellable_reset(c->cancellable); > + g_clear_object(&c->cancellable); > + c->cancellable = g_cancellable_new(); > start_demux(self); > } else { > g_cancellable_cancel(c->cancellable); > @@ -529,7 +530,6 @@ static void spice_webdav_channel_init(SpiceWebdavChannel *channel) > > channel->priv = c; > c->stream = spice_vmc_stream_new(SPICE_CHANNEL(channel)); > - c->cancellable = g_cancellable_new(); > c->clients = g_hash_table_new_full(g_int64_hash, g_int64_equal, > NULL, client_remove_unref); > c->demux.buf = g_malloc0(MAX_MUX_SIZE); > -- > 2.9.2 > > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/spice-devel _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel