PAM merge from FreeBSD

Dag-Erling Smorgrav des at ofug.org
Mon Feb 3 20:21:54 EST 2003


Damien Miller <djm at mindrot.org> writes:
> Dag-Erling Smorgrav wrote:
> > BTW, the patches Damien posted contain a bug which breaks ssh1 in some
> > cases.
> Was this the extra empty query in auth-chall.c? If so, I sent an
> updated patch which does:

Not just that, there's a problem in the monitor code as well.
Sometimes the monitor is not aware that authentication succeeded -
although it's possible that the auth-chall.c patch hides this bug.

I see your code is nearly identical to mine, except for variable names
and the use of goto.  Great minds think alike :)

DES
-- 
Dag-Erling Smorgrav - des at ofug.org




More information about the openssh-unix-dev mailing list