diff options
| author | Ray Strode <rstrode@redhat.com> | 2012-09-06 13:51:45 -0400 |
|---|---|---|
| committer | Ray Strode <rstrode@redhat.com> | 2012-09-06 13:55:12 -0400 |
| commit | 360772d6c9f1b3e099c19271ca12443d72fa799d (patch) | |
| tree | ff76dd5a864c9844c12e7b5a20b94fb0f542ed7a /daemon/gdm-simple-slave.c | |
| parent | aeb5c6fa4e46653efda85a7341abee5a0565fe97 (diff) | |
| download | gdm-360772d6c9f1b3e099c19271ca12443d72fa799d.tar.gz | |
slave: don't prematurely set is_running to FALSE
When the session ends we
1) set the session id to NULL
2) set is_running to FALSE
3) call gdm_slave_stop which
4) runs the PostSesssion script if is_running is true
5) sets is_running to FALSE
4) will always fail because of 2, and 2 is unnecessary because of 5.
This commit defers settings is_running to false until gdm_session_stop.
Spotted by Gert Kulyk
https://bugzilla.gnome.org/show_bug.cgi?id=681888
Diffstat (limited to 'daemon/gdm-simple-slave.c')
| -rw-r--r-- | daemon/gdm-simple-slave.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/daemon/gdm-simple-slave.c b/daemon/gdm-simple-slave.c index 2af76c76..20671c8c 100644 --- a/daemon/gdm-simple-slave.c +++ b/daemon/gdm-simple-slave.c @@ -229,7 +229,6 @@ on_session_exited (GdmSession *session, int exit_code, GdmSimpleSlave *slave) { - slave->priv->session_is_running = FALSE; g_object_set (GDM_SLAVE (slave), "session-id", NULL, NULL); g_debug ("GdmSimpleSlave: session exited with code %d\n", exit_code); @@ -241,7 +240,6 @@ on_session_died (GdmSession *session, int signal_number, GdmSimpleSlave *slave) { - slave->priv->session_is_running = FALSE; g_object_set (GDM_SLAVE (slave), "session-id", NULL, NULL); g_debug ("GdmSimpleSlave: session died with signal %d, (%s)", |