[virt-tools-list] [virt-viewer][PATCH] events: Don't leak GIOChannel when destroying IO handle

[Fabiano Fidêncio]

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