Blame SOURCES/0033-manager-correct-display-confusion.patch

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