diff options
author | George Lebl <jirka@5z.com> | 2002-08-23 08:29:49 +0000 |
---|---|---|
committer | George Lebl <jirka@src.gnome.org> | 2002-08-23 08:29:49 +0000 |
commit | ce50f985f0ca818e8f132ff3afcb23c3ea37609f (patch) | |
tree | cc1ca615c047a83fe901b554482133245392b55d /TODO | |
parent | 59dea77978a334b783794d1b673c752a3d7d0aab (diff) | |
download | gdm-ce50f985f0ca818e8f132ff3afcb23c3ea37609f.tar.gz |
fix subtle possible race that could kill another process and simplify code
Fri Aug 23 01:09:36 2002 George Lebl <jirka@5z.com>
* gui/gdmlogin.c: fix subtle possible race that could kill another
process and simplify code in the meantime. Ignore SIGCHLD, and
don't wait for the background process until we want to kill it,
that may leave a zombie for a little bit if the process dies early,
but who cares.
* daemon/display.c, daemon/gdm.c, daemon/misc.c, daemon/server.c,
daemon/slave.c: Fix many many very minor races but such that
could conceivably take things down (a kill (0,...) "could"
very theoretically occur previously). Fix some worse but still
not very likely races by using the push/pop of sigchld block in
places where we blocked by hand. Ensure all processes are always
killed on slave exit (probably doing things redundantly in some
cases). Fix some possible cases of syslog from a signal which
does bad things to our karma sometimes. When the server dies
make sure we don't do anything with the display (we null the d->dsp
var) to make sure we don't reenter the xioerror_handler. The
extra_process uses 0 and not -1 as the no value everywhere.
Make sure kills are never called with -1 (probably just anal,
but one never knows). Do setsid and the init in daemonize
on -nodaemon as well. Do setsid for greeter/chooser/config.
Diffstat (limited to 'TODO')
0 files changed, 0 insertions, 0 deletions