[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#932047: lightdm: greeter session support for elogind



Package: lightdm
Severity: normal
Tags: patch

Yves-Alexis,

Many thanks for updating lightdm's dependencies to use the new logind virtual
packages in closing #922160.

However, the necessary adjustments to the PAM configuration for lightdm-greeter
are still outstanding.

My testing indicates that in order for lightdm greeter's Suspend, Hibernate,
Restart and Shut Down buttons to be enabled, the greeter itself needs to
register a logind session.

For elogind based systems this could be implemented by adding

 session   optional pam_elogind.so

to /etc/pam.d/lightdm-greeter

Or, alternatively and perhaps better for the future, whichever logind
implementation is enabled through pam-auth could be used by sourcing the PAM
common-session.

Patches implementing both of these approaches are attached.

Many thanks,

Mark
1 file changed, 5 insertions(+)
debian/patches/05_debianize-pam-files.patch | 5 +++++

modified   debian/patches/05_debianize-pam-files.patch
@@ -60,6 +60,11 @@
  
  # Always let the greeter start without authentication
  auth      required pam_permit.so
+@@ -15,3 +16,4 @@
+ # Setup session
+ session   required pam_unix.so
+ session   optional pam_systemd.so
++session   optional pam_elogind.so
 --- a/data/pam/lightdm-autologin
 +++ b/data/pam/lightdm-autologin
 @@ -1,20 +1,37 @@

[back]
1 file changed, 7 insertions(+)
debian/patches/05_debianize-pam-files.patch | 7 +++++++

modified   debian/patches/05_debianize-pam-files.patch
@@ -60,6 +60,13 @@
  
  # Always let the greeter start without authentication
  auth      required pam_permit.so
+@@ -13,5 +14,4 @@
+ password  required pam_deny.so
+ 
+ # Setup session
+-session   required pam_unix.so
+-session   optional pam_systemd.so
++@include common-session
 --- a/data/pam/lightdm-autologin
 +++ b/data/pam/lightdm-autologin
 @@ -1,20 +1,37 @@

[back]

Reply to: