bug-cvs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Compile error in current CVS, src/server.c:5500


From: Derek Robert Price
Subject: Re: Compile error in current CVS, src/server.c:5500
Date: Mon, 21 Jul 2003 14:23:42 -0400
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624 Netscape/7.1

Brian Murphy wrote:

Hi,

Here is the fix. I didn't like the use of a fake error EACCES
in this situation, I prefer 0. EACCES refers to file access
failures not authentication / login failures as far as I'm aware.


What, exactly, is wrong with calling pam_strerror() to get an error message from PAM for pam_end() errors and printing that, as Mark mentioned? For that matter, why isn't this being done for returns other than PAM_SUCCESS in the preceding calls to pam_start(), pam_authenticate(), and pam_acct_mgmt()?

Derek

--
               *8^)

Email: derek@ximbiot.com

Get CVS support at <http://ximbiot.com>!
--
If thou dost marry, I'll give thee this plague for
thy dowry:  be thou as chaste as ice, as pure as snow,
thou shalt not escape calumny.  Get thee to a nunnery.
Go, farewell.  Or if thou wilt needs marry, marry a fool.
For wise men know well enough what monsters you
make of them.  To a nunnery, go, and quickly, too.  Fare-
well.

    - Hamlet, Act III, Scene 1, Lines 135-141






reply via email to

[Prev in Thread] Current Thread [Next in Thread]