severity 361163 normal tags 361163 = pending merge 361163 361089 thanks On Fri, Apr 07, 2006 at 02:02:31AM +0200, Michael Biebl wrote: > Denis Barbier wrote: > [..] > > diff -ur debian.orig/kdm-np.pam debian/kdm-np.pam > > --- debian.orig/kdm-np.pam 2006-04-07 00:50:17.000000000 +0200 > > +++ debian/kdm-np.pam 2006-04-07 01:07:13.000000000 +0200 > > @@ -13,6 +13,7 @@ > > > > auth required pam_nologin.so > > auth required pam_env.so > > +auth required pam_env.so envfile=/etc/default/locale > > Should be "session" and not "auth" imho as I already posted in #361089.
Oops I did not see your bugreport, merging them now. pam_env is an auth module, see pam-0.79/Linux-PAM/modules/pam_env/README # The original idea for this came from Andrew G. Morgan ... #<quote> # Mmm. Perhaps you might like to write a pam_env module that reads a # default environment from a file? I can see that as REALLY # useful... Note it would be an "auth" module that returns PAM_IGNORE # for the auth part and sets the environment returning PAM_SUCCESS in # the setcred function... #</quote> It is true that shadow has it wrong, but in practice I do not know whether it does really matter. Denis -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
