From: Lennart Poettering <lennart@poettering.net>
Date: Wed, 9 Aug 2017 17:04:36 +0000 (+0200)
Subject: pam_logind: skip leading /dev/ from PAM_TTY field before passing it on
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=63e7758940d27499845c3e45b61e39c47efdc8c3;p=elogind.git

pam_logind: skip leading /dev/ from PAM_TTY field before passing it on

Apparently, PAM documents that the PAM_TTY should come with a /dev
prefix, but we don't expect it so far, except that Wayland ends up
setting it after all, the way the docs suggest. Hence, let's simply drop
the /dev prefix if it is there.

Fixes: #6516
---

diff --git a/src/login/pam_elogind.c b/src/login/pam_elogind.c
index 3af3487f5..5300e218b 100644
--- a/src/login/pam_elogind.c
+++ b/src/login/pam_elogind.c
@@ -46,6 +46,7 @@
 #include "strv.h"
 #include "terminal-util.h"
 #include "util.h"
+#include "path-util.h"
 
 static int parse_argv(
                 pam_handle_t *handle,
@@ -339,7 +340,9 @@ _public_ PAM_EXTERN int pam_sm_open_session(
                 type ="tty";
                 class = "user";
                 tty = NULL;
-        }
+        } else
+                /* Chop off leading /dev prefix that some clients specify, but others do not. */
+                tty = skip_dev_prefix(tty);
 
         /* If this fails vtnr will be 0, that's intended */
         if (!isempty(cvtnr))