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-09-01 00:54:45
Message-ID: Pine.LNX.4.21.0108311953240.14539-200000@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:

> 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.

See attached revised patch. (I'm sure the configure.in stuff can be done
right/better, I'm just not enough of a autoconf guru to know what to
change it to.)

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

Attachment Content-Type Size
pgsql-diff.log text/plain 13.9 KB

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Thomas Lockhart 2001-09-01 01:16:05 Re: INTERVAL type: SQL92 implementation
Previous Message Dave Cramer 2001-08-31 19:00:27 Re: Read transactions don't work on 7.0.x db's