[glib/mcatanzaro/#2221] Fix race in socketclient-slow test




commit 2131dff47f22213f153a4018bb8b9c402437271b
Author: Michael Catanzaro <mcatanzaro gnome org>
Date:   Mon Oct 19 08:57:46 2020 -0500

    Fix race in socketclient-slow test
    
    This test ensures that g_socket_client_connect_to_host_async() fails if
    it is cancelled, but it's not cancelled until after 1 millisecond. Our
    CI testers are hitting that race window, and Milan is able to reproduce
    the crash locally as well. Switching it from 1ms to 0ms is enough to
    avoid the crash, but let's be extra-cautious and use G_PRIORITY_HIGH
    as well, for good measure.
    
    Fixes #2221

 gio/tests/gsocketclient-slow.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
---
diff --git a/gio/tests/gsocketclient-slow.c b/gio/tests/gsocketclient-slow.c
index 34410f4cf..5d5a5c38f 100644
--- a/gio/tests/gsocketclient-slow.c
+++ b/gio/tests/gsocketclient-slow.c
@@ -124,7 +124,7 @@ test_happy_eyeballs_cancel_delayed (void)
   client = g_socket_client_new ();
   cancel = g_cancellable_new ();
   g_socket_client_connect_to_host_async (client, "localhost", port, cancel, on_connected_cancelled, loop);
-  g_timeout_add (1, (GSourceFunc) on_timer, cancel);
+  g_timeout_add_full (G_PRIORITY_HIGH, 0, (GSourceFunc) on_timer, cancel, NULL);
   g_signal_connect (client, "event", G_CALLBACK (on_event), &got_completed_event);
   g_main_loop_run (loop);
 


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