Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 08 May 2001 23:19:44 -0400
From:      "Brian F. Feldman" <green@FreeBSD.org>
To:        Alfred Perlstein <bright@wintelcom.net>
Cc:        Brian Feldman <green@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/crypto/openssh auth-pam.c 
Message-ID:  <200105090319.f493JiE24936@green.bikeshed.org>
In-Reply-To: Message from Alfred Perlstein <bright@wintelcom.net>  of "Tue, 08 May 2001 20:00:19 PDT." <20010508200018.S18676@fw.wintelcom.net> 

next in thread | previous in thread | raw e-mail | index | archive | help
Alfred Perlstein <bright@wintelcom.net> wrote:
> * Brian Feldman <green@FreeBSD.org> [010508 15:30] wrote:
> > green       2001/05/08 15:30:18 PDT
> > 
> >   Modified files:
> >     crypto/openssh       auth-pam.c 
> >   Log:
> >   Since PAM is broken, let pam_setcred() failure be non-fatal.
> 
> That could be pretty bad if auth_pam_password fails with pam.
> 
> here's what I think is a better fix.
> 
> I sync'd the code up a bit with the "portable ssh" project.
> 
> please review as soon as you can as i'm nervous about the 
> missing check for 'was_authenticated'

It looks good.  No objection here :)

-- 
 Brian Fundakowski Feldman           \  FreeBSD: The Power to Serve!  /
 green@FreeBSD.org                    `------------------------------'



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200105090319.f493JiE24936>