Bug#361163: Please let /etc/pam.d/kdm read locale variables from /etc/default/locale

[Denis Barbier <barbier@linuxfr.org>]

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