From 63e7758940d27499845c3e45b61e39c47efdc8c3 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Wed, 9 Aug 2017 19:04:36 +0200 Subject: [PATCH] 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 --- src/login/pam_elogind.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) 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)) -- 2.30.2