From: Alan Jenkins <alan.christopher.jenkins@gmail.com>
Date: Sat, 3 Mar 2018 18:37:50 +0000 (+0000)
Subject: login: fix FDNAME in call to sd_pid_notify_with_fds()
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=49737613227ddfe7191797965c8a6d005d890bb8;p=elogind.git

login: fix FDNAME in call to sd_pid_notify_with_fds()

$ git grep FDNAME
logind-session-device.c: ... "FDNAME=session-", sd->session->id);
logind-session-device.c: ... "FDNAME=session", sd->session->id);

Oops.

Fixes #8343.  Or at least a more minimal reproducer.  Xorg still
dies when logind is restarted, but the Xorg message says this
is entirely deliberate.

(This could also be the reason I hit #8035, instead of the race
condition I originally suggested).
---

diff --git a/src/login/logind-session-device.c b/src/login/logind-session-device.c
index e127a3e99..9132d0905 100644
--- a/src/login/logind-session-device.c
+++ b/src/login/logind-session-device.c
@@ -530,7 +530,7 @@ int session_device_save(SessionDevice *sd) {
                 return 0;
 
         m = strjoina("FDSTORE=1\n"
-                     "FDNAME=session", sd->session->id);
+                     "FDNAME=session-", sd->session->id);
 
         r = sd_pid_notify_with_fds(0, false, m, &sd->fd, 1);
         if (r < 0)