|
 |
698723 |
From 35f9a7f8f4e8917725349fe764706658c02537ca Mon Sep 17 00:00:00 2001
|
|
|
698723 |
From: Lennart Poettering <lennart@poettering.net>
|
|
|
698723 |
Date: Mon, 6 Aug 2018 19:02:29 +0200
|
|
|
698723 |
Subject: [PATCH] logind: never elect a session that is stopping as display
|
|
|
698723 |
|
|
|
698723 |
(cherry picked from commit 04857cd801022d9f9933efb484c6253572f09870)
|
|
|
698723 |
|
|
|
698723 |
Related: #1642460
|
|
|
698723 |
---
|
|
|
698723 |
src/login/logind-user.c | 10 ++++------
|
|
|
698723 |
1 file changed, 4 insertions(+), 6 deletions(-)
|
|
|
698723 |
|
|
|
698723 |
diff --git a/src/login/logind-user.c b/src/login/logind-user.c
|
|
|
698723 |
index 35b2ca5489..3e4c99bdbd 100644
|
|
|
698723 |
--- a/src/login/logind-user.c
|
|
|
698723 |
+++ b/src/login/logind-user.c
|
|
|
698723 |
@@ -613,11 +613,10 @@ int user_kill(User *u, int signo) {
|
|
|
698723 |
}
|
|
|
698723 |
|
|
|
698723 |
static bool elect_display_filter(Session *s) {
|
|
|
698723 |
- /* Return true if the session is a candidate for the user’s ‘primary
|
|
|
698723 |
- * session’ or ‘display’. */
|
|
|
698723 |
+ /* Return true if the session is a candidate for the user’s ‘primary session’ or ‘display’. */
|
|
|
698723 |
assert(s);
|
|
|
698723 |
|
|
|
698723 |
- return (s->class == SESSION_USER && !s->stopping);
|
|
|
698723 |
+ return s->class == SESSION_USER && s->started && !s->stopping;
|
|
|
698723 |
}
|
|
|
698723 |
|
|
|
698723 |
static int elect_display_compare(Session *s1, Session *s2) {
|
|
|
698723 |
@@ -663,9 +662,8 @@ void user_elect_display(User *u) {
|
|
|
698723 |
|
|
|
698723 |
assert(u);
|
|
|
698723 |
|
|
|
698723 |
- /* This elects a primary session for each user, which we call
|
|
|
698723 |
- * the "display". We try to keep the assignment stable, but we
|
|
|
698723 |
- * "upgrade" to better choices. */
|
|
|
698723 |
+ /* This elects a primary session for each user, which we call the "display". We try to keep the assignment
|
|
|
698723 |
+ * stable, but we "upgrade" to better choices. */
|
|
|
698723 |
log_debug("Electing new display for user %s", u->name);
|
|
|
698723 |
|
|
|
698723 |
LIST_FOREACH(sessions_by_user, s, u->sessions) {
|