chiark / gitweb /
login: fix FDNAME in call to sd_pid_notify_with_fds()
authorAlan Jenkins <alan.christopher.jenkins@gmail.com>
Sat, 3 Mar 2018 18:37:50 +0000 (18:37 +0000)
committerSven Eden <yamakuzure@gmx.net>
Wed, 30 May 2018 05:59:14 +0000 (07:59 +0200)
$ 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).

src/login/logind-session-device.c

index e127a3e99cfc95d98ee40f8b291e6eef62a3ef4d..9132d0905211d96386f051897ab6f7042c58c61e 100644 (file)
@@ -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)