Re: Kerberos authentication, Active Directory, and PostgreSQL

Peter Eisentraut wrote:
> On Mon, 2009-10-12 at 20:42 -0400, Turner, Ian wrote:
> > --- postgresql-8.4-8.4.1/src/backend/libpq/auth.c 2009-06-25 12:30:08.000000000 +0100
> > +++ postgresql-8.4-8.4.1-fixed/src/backend/libpq/auth.c 2009-09-15 20:27:01.000000000 +0100
> > @@ -166,6 +166,8 @@
> > #endif
> >
> > static int pg_GSS_recvauth(Port *port);
> > +
> > +#define GSS_MAX_TOKEN_LENGTH (32767)
> > #endif /* ENABLE_GSS */
> >
> >
>
> A small wish in case we go with this: The constant should be named
> something like PG_...; otherwise it looks like we are defining or
> overriding an official symbol from the GSS API.

Agreed. In my first glance at the patch I thought GSS_MAX_TOKEN_LENGTH
was defined in a system include file.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +