Re: PAM Authentication Patch, take one...

From: "Dominic J(dot) Eidson" <sauron(at)the-infinite(dot)org>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: PAM Authentication Patch, take one...
Date: 2001-08-28 20:58:27
Message-ID: Pine.LNX.4.21.0108281558010.26153-100000@morannon.the-infinite.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

On Tue, 28 Aug 2001, Peter Eisentraut wrote:

> Dominic J. Eidson writes:
>
> > Attached is the new and revised version of the PAM authentication patch.
> >
> > This will only send a password request to the frontend, if a password is
> > required.
>
> pam_strerror() should be used a few more times, rather than just saying
> "Error!". Also, the configure.in snippet seems wrong. You add
> -I$pam_prefix/include/security to $INCLUDES and then you #include
> <security/pam_appl.h>. This whole thing is probably unnecessary, since
> PAM is a system library on the systems where it exists, so the headers
> and libraries are found automatically, unlike OpenSSL and
> Kerberos.

Will fix all of this and resubmit.

--
Dominic J. Eidson
"Baruk Khazad! Khazad ai-menu!" - Gimli
-------------------------------------------------------------------------------
http://www.the-infinite.org/ http://www.the-infinite.org/~dominic/

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Jeroen van Vianen 2001-08-28 21:20:19 Patch for broken JDBC's getColumn()
Previous Message Peter Eisentraut 2001-08-28 19:21:57 Re: PAM Authentication Patch, take one...