Hello community,
here is the log from the commit of package gdm for openSUSE:Factory checked in
at 2020-04-18 00:27:05
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/gdm (Old)
and /work/SRC/openSUSE:Factory/.gdm.new.2738 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "gdm"
Sat Apr 18 00:27:05 2020 rev:225 rq:794355 version:3.34.1
Changes:
--------
--- /work/SRC/openSUSE:Factory/gdm/gdm.changes 2020-03-11 18:32:55.226919802
+0100
+++ /work/SRC/openSUSE:Factory/.gdm.new.2738/gdm.changes 2020-04-18
00:27:08.341643382 +0200
@@ -1,0 +2,7 @@
+Fri Apr 10 09:12:37 UTC 2020 - Yifan Jiang <yfji...@suse.com>
+
+- Add gdm-look-for-session-based-on-pid-first.patch: Look for
+ session based on pid first, then fall back to the uid based
+ approach (bsc#1159950, glgo#GNOME/gdm#526).
+
+-------------------------------------------------------------------
New:
----
gdm-look-for-session-based-on-pid-first.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ gdm.spec ++++++
--- /var/tmp/diff_new_pack.nP2mKz/_old 2020-04-18 00:27:11.593650116 +0200
+++ /var/tmp/diff_new_pack.nP2mKz/_new 2020-04-18 00:27:11.597650124 +0200
@@ -65,6 +65,8 @@
Patch14: gdm-switch-user-tty7.patch
# PATCH-FIX-UPSTREAM gdm-disable-wayland-on-mgag200-chipsets.patch bsc#1162888
glgo#GNOME/mutter#57 qk...@suse.com -- Disable Wayland on mgag200 chipsets
Patch15: gdm-disable-wayland-on-mgag200-chipsets.patch
+# PATCH-FIX-UPSTREAM gdm-look-for-session-based-on-pid-first.patch bsc#1159950
glgo#GNOME/gdm#526 yfji...@suse.com -- Look for session based on pid first,
then fall back to the uid based approach
+Patch16: gdm-look-for-session-based-on-pid-first.patch
### NOTE: Keep please SLE-only patches at bottom (starting on 1000).
# PATCH-FIX-SLE gdm-disable-gnome-initial-setup.patch bnc#1067976
qz...@suse.com -- Disable gnome-initial-setup runs before gdm, g-i-s will only
serve for CJK people to choose the input-method after login.
Patch1000: gdm-disable-gnome-initial-setup.patch
@@ -221,6 +223,7 @@
%endif
%patch14 -p1
%patch15 -p1
+%patch16 -p1
# SLE-only patches start at 1000
%if !0%{?is_opensuse}
++++++ gdm-look-for-session-based-on-pid-first.patch ++++++
>From 82c3c9bbe924c43e93e74cd622207a54bc44962c Mon Sep 17 00:00:00 2001
From: Benjamin Berg <bb...@redhat.com>
Date: Thu, 23 Jan 2020 14:32:38 +0100
Subject: [PATCH] manager: Try looking up session based on PID first
Unfortunately, GDM may be running multiple greeters, and each greeter is
currently using the same user. So while in a lot of setups each user
should only have one graphical session and also only one DBus session
bus, this is not true for the gdm greeter.
Lacking another solution (e.g. separate users), we need to be able to
correctly lookup the session information for all greeter instances. We
can do so by using sd_pid_get_session and using this information is safe
if it does return something.
See: #526
---
common/gdm-common.c | 30 ++++++++++++++++++++++++++----
common/gdm-common.h | 7 ++++---
daemon/gdm-manager.c | 2 +-
libgdm/gdm-user-switching.c | 2 +-
4 files changed, 32 insertions(+), 9 deletions(-)
diff --git a/common/gdm-common.c b/common/gdm-common.c
index 41bdb389..d13cf618 100644
--- a/common/gdm-common.c
+++ b/common/gdm-common.c
@@ -497,7 +497,7 @@ goto_login_session (GDBusConnection *connection,
* since the data allocated is from libsystemd-logind, which
* does not use GLib's g_malloc (). */
- if (!gdm_find_display_session_for_uid (getuid (), &our_session,
&local_error)) {
+ if (!gdm_find_display_session (0, getuid (), &our_session,
&local_error)) {
g_propagate_prefixed_error (error, local_error, _("Could not
identify the current session: "));
return FALSE;
@@ -898,16 +898,38 @@ _systemd_session_is_active (const char *session_id)
}
gboolean
-gdm_find_display_session_for_uid (const uid_t uid,
- char **out_session_id,
- GError **error)
+gdm_find_display_session (int pid,
+ const uid_t uid,
+ char **out_session_id,
+ GError **error)
{
char *local_session_id = NULL;
g_auto(GStrv) sessions = NULL;
int n_sessions;
+ int res;
g_return_val_if_fail (out_session_id != NULL, FALSE);
+ /* First try to look up the session using the pid. We need this
+ * at least for the greeter, because it currently runs multiple
+ * sessions under the same user.
+ * See also commit 2b52d8933c8ab38e7ee83318da2363d00d8c5581
+ * which added an explicit dbus-run-session for all but seat0.
+ */
+ res = sd_pid_get_session (pid, &local_session_id);
+ if (res >= 0) {
+ g_debug ("Found session %s for PID %d, using",
local_session_id, pid);
+
+ *out_session_id = g_strdup (local_session_id);
+ g_free (local_session_id);
+
+ return TRUE;
+ } else {
+ if (res != -ENODATA)
+ g_warning ("Failed to retrieve session information for
pid %d: %s",
+ pid, strerror (-res));
+ }
+
g_debug ("Finding a graphical session for user %d", uid);
n_sessions = sd_uid_get_sessions (uid,
diff --git a/common/gdm-common.h b/common/gdm-common.h
index 5c3fe137..3d037d68 100644
--- a/common/gdm-common.h
+++ b/common/gdm-common.h
@@ -51,9 +51,10 @@ int gdm_wait_on_and_disown_pid (int pid,
int gdm_signal_pid (int pid,
int signal);
-gboolean gdm_find_display_session_for_uid (const uid_t uid,
- char **out_session_id,
- GError **error);
+gboolean gdm_find_display_session (int pid,
+ const uid_t uid,
+ char **out_session_id,
+ GError **error);
gboolean gdm_get_pwent_for_name (const char *name,
struct passwd **pwentp);
diff --git a/daemon/gdm-manager.c b/daemon/gdm-manager.c
index 907eca37..d8dfa843 100644
--- a/daemon/gdm-manager.c
+++ b/daemon/gdm-manager.c
@@ -501,7 +501,7 @@ get_display_and_details_for_bus_sender (GdmManager
*self,
goto out;
}
- ret = gdm_find_display_session_for_uid (caller_uid, &session_id,
&error);
+ ret = gdm_find_display_session (pid, caller_uid, &session_id, &error);
if (!ret) {
g_debug ("GdmManager: Unable to find display session for uid
%d: %s",
diff --git a/libgdm/gdm-user-switching.c b/libgdm/gdm-user-switching.c
index 3a33fcbb..20235fd8 100644
--- a/libgdm/gdm-user-switching.c
+++ b/libgdm/gdm-user-switching.c
@@ -203,7 +203,7 @@ goto_login_session (GDBusConnection *connection,
/* Note that we mostly use free () here, instead of g_free ()
* since the data allocated is from libsystemd-logind, which
* does not use GLib's g_malloc (). */
- if (!gdm_find_display_session_for_uid (getuid (), &our_session,
&local_error)) {
+ if (!gdm_find_display_session (0, getuid (), &our_session,
&local_error)) {
g_propagate_prefixed_error (error, local_error, _("Could not
identify the current session: "));
return FALSE;
--
2.24.1
