Solaris and Latest snapshot (2001-02-21)

[mouring at etoh.eviladmin.org]

On Tue, 20 Feb 2001, Devon Bleak wrote:

> all i use is protocol 2...  someone else will have to verify v1 for
> sure, but i believe in the original email about this issue the author
> said that it was only broken when they were forcing version 2.
> 
> anyways, i tested the snapshots from around the 15th (first email on the
> issue appeared) on bass.directhit.com and:
> 20010215 works.
> 20010216 is broken.
> 
> devon
> 
There is only two sets of pam patches around that date.  One is a
suggestion to clean up PAM space.  Which I can't see any fault.  And this
one which stated it was for solaris.  Can you apply this reverse and see
if this gets us closer to fixing this?  Or if the problem still exists?

- Ben

--- session.c	Tue Feb 20 20:31:55 2001
+++ ../openssh/session.c	Sun Feb 18 13:03:29 2001
@@ -1017,6 +1017,10 @@
 #endif /* WITH_IRIX_ARRAY */
 #endif /* WITH_IRIX_JOBS */
 
+#ifdef USE_PAM
+	do_pam_session(pw->pw_name, ttyname);
+	do_pam_setcred();
+#endif /* USE_PAM */
 
 	/* login(1) is only called if we execute the login shell */
 	if (options.use_login && command != NULL)
@@ -1142,11 +1146,6 @@
 #ifdef HAVE_LOGIN_CAP
 	shell = login_getcapstr(lc, "shell", (char *)shell, (char *)shell);
 #endif
-
-#ifdef USE_PAM
-	do_pam_session(pw->pw_name, ttyname);
-	do_pam_setcred();
-#endif /* USE_PAM */
 
 #ifdef AFS
 	/* Try to get AFS tokens for the local cell. */