Blame SOURCES/0023-vdagent-Remove-watch-event-on-vdagent_display_destro.patch

e91d8b
From bcbbea34d93d07d33b767f808ff3adf628b1ea0f Mon Sep 17 00:00:00 2001
e91d8b
From: Victor Toso <victortoso@redhat.com>
e91d8b
Date: Fri, 13 Jan 2023 13:54:06 +0100
e91d8b
Subject: [PATCH] vdagent: Remove watch event on vdagent_display_destroy()
e91d8b
e91d8b
To avoid main loop calling it when GIOChannel for x11 was already
e91d8b
destroyed.
e91d8b
e91d8b
Related: https://bugzilla.redhat.com/show_bug.cgi?id=2145004
e91d8b
Signed-off-by: Victor Toso <victortoso@redhat.com>
e91d8b
---
e91d8b
 src/vdagent/display.c | 6 ++++--
e91d8b
 1 file changed, 4 insertions(+), 2 deletions(-)
e91d8b
e91d8b
diff --git a/src/vdagent/display.c b/src/vdagent/display.c
e91d8b
index 790d9ad..602ab65 100644
e91d8b
--- a/src/vdagent/display.c
e91d8b
+++ b/src/vdagent/display.c
e91d8b
@@ -60,6 +60,7 @@ struct VDAgentDisplay {
e91d8b
     UdscsConnection *vdagentd;
e91d8b
     int debug;
e91d8b
     GIOChannel *x11_channel;
e91d8b
+    guint io_watch_source_id;
e91d8b
     VDAgentMutterDBus *mutter;
e91d8b
 };
e91d8b
 
e91d8b
@@ -296,7 +297,8 @@ VDAgentDisplay* vdagent_display_create(UdscsConnection *vdagentd, int debug, int
e91d8b
         return NULL;
e91d8b
     }
e91d8b
 
e91d8b
-    g_io_add_watch(display->x11_channel, G_IO_IN, x11_io_channel_cb, display);
e91d8b
+    display->io_watch_source_id =
e91d8b
+        g_io_add_watch(display->x11_channel, G_IO_IN, x11_io_channel_cb, display);
e91d8b
 
e91d8b
 
e91d8b
     /* Since we are started at the same time as the wm,
e91d8b
@@ -323,7 +325,7 @@ void vdagent_display_destroy(VDAgentDisplay *display, int vdagentd_disconnected)
e91d8b
         return;
e91d8b
     }
e91d8b
 
e91d8b
-
e91d8b
+    g_source_remove(display->io_watch_source_id);
e91d8b
     g_clear_pointer(&display->x11_channel, g_io_channel_unref);
e91d8b
     vdagent_x11_destroy(display->x11, vdagentd_disconnected);
e91d8b
 
e91d8b
-- 
e91d8b
2.39.0
e91d8b