|
 |
2fc437 |
From 6f62469760cfe8b4c798135bba1d9ccaa7f4b0d2 Mon Sep 17 00:00:00 2001
|
|
|
2fc437 |
From: Ray Strode <rstrode@redhat.com>
|
|
|
2fc437 |
Date: Wed, 12 Jul 2017 10:26:37 -0400
|
|
|
2fc437 |
Subject: [PATCH] legacy-display: ensure X server is dead when respawning
|
|
|
2fc437 |
|
|
|
2fc437 |
If the user logs out we may detect the session dying before
|
|
|
2fc437 |
we detect the X server dying. If that happens, then we
|
|
|
2fc437 |
currently don't wait for the X server to exit before starting
|
|
|
2fc437 |
a new X server.
|
|
|
2fc437 |
|
|
|
2fc437 |
This commit makes sure to explicitly kill the X server and wait
|
|
|
2fc437 |
for its death before proceeding.
|
|
|
2fc437 |
---
|
|
|
2fc437 |
daemon/gdm-legacy-display.c | 21 +++++++++++++++++++++
|
|
|
2fc437 |
1 file changed, 21 insertions(+)
|
|
|
2fc437 |
|
|
|
2fc437 |
diff --git a/daemon/gdm-legacy-display.c b/daemon/gdm-legacy-display.c
|
|
|
2fc437 |
index 9c999232..46990ccc 100644
|
|
|
2fc437 |
--- a/daemon/gdm-legacy-display.c
|
|
|
2fc437 |
+++ b/daemon/gdm-legacy-display.c
|
|
|
2fc437 |
@@ -236,51 +236,72 @@ gdm_legacy_display_manage (GdmDisplay *display)
|
|
|
2fc437 |
"server (your graphical environment) "
|
|
|
2fc437 |
"due to an internal error. "
|
|
|
2fc437 |
"Please contact your system administrator "
|
|
|
2fc437 |
"or check your syslog to diagnose. "
|
|
|
2fc437 |
"In the meantime this display will be "
|
|
|
2fc437 |
"disabled. Please restart GDM when "
|
|
|
2fc437 |
"the problem is corrected."));
|
|
|
2fc437 |
gdm_display_unmanage (GDM_DISPLAY (self));
|
|
|
2fc437 |
}
|
|
|
2fc437 |
|
|
|
2fc437 |
g_debug ("GdmDisplay: Started X server");
|
|
|
2fc437 |
|
|
|
2fc437 |
}
|
|
|
2fc437 |
|
|
|
2fc437 |
static void
|
|
|
2fc437 |
gdm_legacy_display_class_init (GdmLegacyDisplayClass *klass)
|
|
|
2fc437 |
{
|
|
|
2fc437 |
GObjectClass *object_class = G_OBJECT_CLASS (klass);
|
|
|
2fc437 |
GdmDisplayClass *display_class = GDM_DISPLAY_CLASS (klass);
|
|
|
2fc437 |
|
|
|
2fc437 |
object_class->constructor = gdm_legacy_display_constructor;
|
|
|
2fc437 |
object_class->finalize = gdm_legacy_display_finalize;
|
|
|
2fc437 |
|
|
|
2fc437 |
display_class->prepare = gdm_legacy_display_prepare;
|
|
|
2fc437 |
display_class->manage = gdm_legacy_display_manage;
|
|
|
2fc437 |
|
|
|
2fc437 |
g_type_class_add_private (klass, sizeof (GdmLegacyDisplayPrivate));
|
|
|
2fc437 |
}
|
|
|
2fc437 |
|
|
|
2fc437 |
static void
|
|
|
2fc437 |
+on_display_status_changed (GdmLegacyDisplay *self)
|
|
|
2fc437 |
+{
|
|
|
2fc437 |
+ int status;
|
|
|
2fc437 |
+
|
|
|
2fc437 |
+ status = gdm_display_get_status (self);
|
|
|
2fc437 |
+
|
|
|
2fc437 |
+ switch (status) {
|
|
|
2fc437 |
+ case GDM_DISPLAY_UNMANAGED:
|
|
|
2fc437 |
+ if (self->priv->server != NULL)
|
|
|
2fc437 |
+ gdm_server_stop (self->priv->server);
|
|
|
2fc437 |
+ break;
|
|
|
2fc437 |
+ default:
|
|
|
2fc437 |
+ break;
|
|
|
2fc437 |
+ }
|
|
|
2fc437 |
+}
|
|
|
2fc437 |
+
|
|
|
2fc437 |
+static void
|
|
|
2fc437 |
gdm_legacy_display_init (GdmLegacyDisplay *legacy_display)
|
|
|
2fc437 |
{
|
|
|
2fc437 |
|
|
|
2fc437 |
legacy_display->priv = GDM_LEGACY_DISPLAY_GET_PRIVATE (legacy_display);
|
|
|
2fc437 |
+
|
|
|
2fc437 |
+ g_signal_connect (legacy_display, "notify::status",
|
|
|
2fc437 |
+ G_CALLBACK (on_display_status_changed),
|
|
|
2fc437 |
+ NULL);
|
|
|
2fc437 |
}
|
|
|
2fc437 |
|
|
|
2fc437 |
GdmDisplay *
|
|
|
2fc437 |
gdm_legacy_display_new (int display_number)
|
|
|
2fc437 |
{
|
|
|
2fc437 |
GObject *object;
|
|
|
2fc437 |
char *x11_display;
|
|
|
2fc437 |
|
|
|
2fc437 |
x11_display = g_strdup_printf (":%d", display_number);
|
|
|
2fc437 |
object = g_object_new (GDM_TYPE_LEGACY_DISPLAY,
|
|
|
2fc437 |
"x11-display-number", display_number,
|
|
|
2fc437 |
"x11-display-name", x11_display,
|
|
|
2fc437 |
NULL);
|
|
|
2fc437 |
g_free (x11_display);
|
|
|
2fc437 |
|
|
|
2fc437 |
return GDM_DISPLAY (object);
|
|
|
2fc437 |
}
|
|
|
2fc437 |
--
|
|
|
2fc437 |
2.13.0
|
|
|
2fc437 |
|