Subject: CVS commit: xsrc/xfree/xc/programs/xdm
To: None <source-changes@NetBSD.org>
From: Ty Sarna <tsarna@netbsd.org>
List: source-changes
Date: 01/25/2006 16:31:21
Module Name:	xsrc
Committed By:	tsarna
Date:		Wed Jan 25 16:31:21 UTC 2006

Modified Files:
	xsrc/xfree/xc/programs/xdm: session.c

Log Message:
Fixes to XDM login handling:

1) PR xsrc/32603 xdm w/ pam_krb5 fails to store tickets

   The pam_setcred call was inside an #ifndef HAS_SETUSERCONTEXT block.
   What that feature was recently disabled, I stopped getting tickets on
   xdm login. Move pam_setcred block outside of and before that block.
   (I believe pam_setcred needs to happen before anyway, as it
   expects to still be root when called)

2) PR xsrc/31376 xdm loses environment settings from PAM

   Setting environment variables from pam needs to happen after
   pam_setcred, since environment variables may be set there
   (and are, by pam_krb5)

3) no PR, mentioned on lists: default path changed

   the setusercontext code was setting the path always, even if no
   path was configured in login.conf, so xdm's default path was
   always overwritten, even if only by _PATH_DEFPATH.

   Now login.conf paths should be used if explicitly set, but xdm's
   default path will be used otherwise.


To generate a diff of this commit:
cvs rdiff -r1.2 -r1.3 xsrc/xfree/xc/programs/xdm/session.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.