Fix races in unix signal dispatch
authorRyan Lortie <desrt@desrt.ca>
Tue, 29 Oct 2013 18:11:28 +0000 (11:11 -0700)
committerColin Walters <walters@verbum.org>
Thu, 2 Jan 2014 00:19:59 +0000 (19:19 -0500)
commit76584e7ae3c3676e6445637f1ad026e5af857938
treebdaba16971ec8e6e31058a59cfaa13ba8bc3ed97
parent97d9138f6c4367bdfe76e3e46871a683c4a2d051
Fix races in unix signal dispatch

Fix some races introduced in be2c7b83c4a9c9d3aa76b1499c27ab19e0f4e470
while keeping the property that multiple handlers for the same unix
signal all get dispatched.

Also fix the behaviour of the source checking for pending signals when
it's created.  No matter what we do here (clear the pending flag or not)
there is something that can go wrong.

If we clear the flag, we may prevent other sources from being
dispatched.  If we don't clear it, we may end up dispatching the same
source twice (if we manage to dispatch it from its own thread before the
GLib worker has a chance to run).

Instead, run the full dispatch procedure when a new source is added.  It
actually doesn't matter what thread this runs in since the lock is held.

https://bugzilla.gnome.org/show_bug.cgi?id=711090
glib/gmain.c