[PATCH libvirt-glib 2/5] tests: fix potential crash

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The idle source is not always removed and may crash next tests

Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxx>
---
 tests/test-events.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/tests/test-events.c b/tests/test-events.c
index 8afb557..0c0e929 100644
--- a/tests/test-events.c
+++ b/tests/test-events.c
@@ -65,6 +65,12 @@ static void watch_cb(int watch, int fd, int events, void *opaque G_GNUC_UNUSED)
     g_message("got event(s) %x on fd %d (watch %d)", events, fd, watch);
 }
 
+static gboolean idle_quit_cb(gpointer user_data G_GNUC_UNUSED)
+{
+    g_main_loop_quit(main_loop);
+
+    return G_SOURCE_REMOVE;
+}
 
 static gboolean test_watch(gpointer user_data G_GNUC_UNUSED)
 {
@@ -81,7 +87,7 @@ static gboolean test_watch(gpointer user_data G_GNUC_UNUSED)
     g_assert_cmpint(removal_status, ==, -1);
     g_idle_add_full(G_PRIORITY_LOW, check_destroyed, &watch_id, NULL);
     g_idle_add_full(G_PRIORITY_LOW,
-                    (GSourceFunc)g_main_loop_quit,
+                    idle_quit_cb,
                     main_loop,
                     NULL);
 
-- 
2.5.0

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]