Re: PAM Authentication Patch, take one...

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: "Dominic J(dot) Eidson" <sauron(at)the-infinite(dot)org>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-patches(at)postgresql(dot)org
Subject: Re: PAM Authentication Patch, take one...
Date: 2001-09-03 21:01:03
Message-ID: 200109032101.f83L13m21623@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches


Your patch has been added to the PostgreSQL unapplied patches list at:

http://candle.pha.pa.us/cgi-bin/pgpatches

I will try to apply it within the next 48 hours.

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

Content-Description:

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Peter Eisentraut 2001-09-03 21:15:15 Re: Patch for pl/tcl Tcl_ExternalToUtf and Tcl_UtfToExternal
Previous Message Bruce Momjian 2001-09-03 20:55:21 Re: libpgtcl.dll