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

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

-- 
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs

Reply via email to