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@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

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