virt_viewer_events_add_handle() creates a GIOChannel in order to watch the fd it was given for changes. virt_viewer_events_remove_handle() is freeing all the resources allocated by virt_viewer_events_add_handle() expect for this GIOChannel. This commit adds the needed g_io_channel_unref() call to virt_viewer_events_remove_handle(). Based on http://libvirt.org/git/?p=libvirt-glib.git;a=commit;h=8e95b8d25a3eee6316aff2f83b0c449aaf10984a Related to: rhbz#1243228 --- src/virt-viewer-events.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/virt-viewer-events.c b/src/virt-viewer-events.c index d36dd3a..6cdc56a 100644 --- a/src/virt-viewer-events.c +++ b/src/virt-viewer-events.c @@ -239,6 +239,10 @@ virt_viewer_events_remove_handle(int watch) data->events = 0; } + g_warn_if_fail(data->channel != NULL); + g_io_channel_unref(data->channel); + data->channel = NULL; + /* since the actual watch deletion is done asynchronously, a handle_update call may * reschedule the watch befure it's fully deleted, that's why we need to mar it as * 'removed' to prevent reuse -- 2.4.4 _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
