|
 |
e0b6b0 |
From 05e49542a9de81731fce68614babe22d437e8fff Mon Sep 17 00:00:00 2001
|
|
|
c90517 |
From: Ray Strode <rstrode@redhat.com>
|
|
|
c90517 |
Date: Mon, 1 Oct 2018 11:05:57 -0400
|
|
|
e0b6b0 |
Subject: [PATCH 33/51] manager: correct display confusion
|
|
|
c90517 |
|
|
|
c90517 |
commit c5c5bf1f reworked autologin and broke it.
|
|
|
c90517 |
|
|
|
c90517 |
This commit addresses the breakage by accessing
|
|
|
c90517 |
the proper display variable.
|
|
|
c90517 |
|
|
|
c90517 |
Closes https://gitlab.gnome.org/GNOME/gdm/issues/426
|
|
|
c90517 |
---
|
|
|
c90517 |
daemon/gdm-manager.c | 2 +-
|
|
|
c90517 |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
c90517 |
|
|
|
c90517 |
diff --git a/daemon/gdm-manager.c b/daemon/gdm-manager.c
|
|
|
c90517 |
index 228cec6ff..4c81dac7f 100644
|
|
|
c90517 |
--- a/daemon/gdm-manager.c
|
|
|
c90517 |
+++ b/daemon/gdm-manager.c
|
|
|
c90517 |
@@ -1711,61 +1711,61 @@ on_start_user_session (StartUserSessionOperation *operation)
|
|
|
c90517 |
if (doing_initial_setup) {
|
|
|
c90517 |
g_debug ("GdmManager: closing down initial setup display");
|
|
|
c90517 |
gdm_display_stop_greeter_session (display);
|
|
|
c90517 |
gdm_display_unmanage (display);
|
|
|
c90517 |
gdm_display_finish (display);
|
|
|
c90517 |
} else {
|
|
|
c90517 |
g_debug ("GdmManager: session has its display server, reusing our server for another login screen");
|
|
|
c90517 |
}
|
|
|
c90517 |
|
|
|
c90517 |
/* The user session is going to follow the session worker
|
|
|
c90517 |
* into the new display. Untie it from this display and
|
|
|
c90517 |
* create a new session for a future user login. */
|
|
|
c90517 |
allowed_uid = gdm_session_get_allowed_user (operation->session);
|
|
|
c90517 |
g_object_set_data (G_OBJECT (display), "gdm-user-session", NULL);
|
|
|
c90517 |
g_object_set_data (G_OBJECT (operation->session), "gdm-display", NULL);
|
|
|
c90517 |
create_user_session_for_display (operation->manager, display, allowed_uid);
|
|
|
c90517 |
|
|
|
c90517 |
/* Give the user session a new display object for bookkeeping purposes */
|
|
|
c90517 |
create_display_for_user_session (operation->manager,
|
|
|
c90517 |
operation->session,
|
|
|
c90517 |
session_id);
|
|
|
c90517 |
|
|
|
c90517 |
if ((g_strcmp0 (operation->service_name, "gdm-autologin") == 0) &&
|
|
|
c90517 |
!gdm_session_client_is_connected (operation->session)) {
|
|
|
c90517 |
/* remove the unused prepared greeter display since we're not going
|
|
|
c90517 |
* to have a greeter */
|
|
|
c90517 |
gdm_display_store_remove (self->priv->display_store, display);
|
|
|
c90517 |
g_object_unref (display);
|
|
|
c90517 |
|
|
|
c90517 |
self->priv->automatic_login_display = g_object_get_data (G_OBJECT (operation->session), "gdm-display");
|
|
|
c90517 |
- g_object_add_weak_pointer (G_OBJECT (display), (gpointer *) &self->priv->automatic_login_display);
|
|
|
c90517 |
+ g_object_add_weak_pointer (G_OBJECT (self->priv->automatic_login_display), (gpointer *) &self->priv->automatic_login_display);
|
|
|
c90517 |
}
|
|
|
c90517 |
}
|
|
|
c90517 |
|
|
|
c90517 |
start_user_session (operation->manager, operation);
|
|
|
c90517 |
|
|
|
c90517 |
out:
|
|
|
c90517 |
return G_SOURCE_REMOVE;
|
|
|
c90517 |
}
|
|
|
c90517 |
|
|
|
c90517 |
static void
|
|
|
c90517 |
queue_start_user_session (GdmManager *manager,
|
|
|
c90517 |
GdmSession *session,
|
|
|
c90517 |
const char *service_name)
|
|
|
c90517 |
{
|
|
|
c90517 |
StartUserSessionOperation *operation;
|
|
|
c90517 |
|
|
|
c90517 |
operation = g_slice_new0 (StartUserSessionOperation);
|
|
|
c90517 |
operation->manager = manager;
|
|
|
c90517 |
operation->session = g_object_ref (session);
|
|
|
c90517 |
operation->service_name = g_strdup (service_name);
|
|
|
c90517 |
|
|
|
c90517 |
operation->idle_id = g_idle_add ((GSourceFunc) on_start_user_session, operation);
|
|
|
c90517 |
g_object_set_data (G_OBJECT (session), "start-user-session-operation", operation);
|
|
|
c90517 |
}
|
|
|
c90517 |
|
|
|
c90517 |
static void
|
|
|
c90517 |
start_user_session_if_ready (GdmManager *manager,
|
|
|
c90517 |
GdmSession *session,
|
|
|
c90517 |
const char *service_name)
|
|
|
c90517 |
{
|
|
|
c90517 |
--
|
|
|
e0b6b0 |
2.27.0
|
|
|
c90517 |
|