Re: [HACKERS] PostgreSQL libraries - PThread Support, but

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Kurt Roeckx <Q(at)ping(dot)be>
Cc: Lee Kindness <lkindness(at)csl(dot)co(dot)uk>, pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org
Subject: Re: [HACKERS] PostgreSQL libraries - PThread Support, but
Date: 2003-06-14 19:18:53
Message-ID: 200306141918.h5EJIr301633@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Kurt Roeckx wrote:
> On Sat, Jun 14, 2003 at 02:20:44PM -0400, Bruce Momjian wrote:
> >
> > This patch handles two more gethostbyname calls.
> >
> [...]
> > diff -c -c -r1.24 fe-secure.c
> > *** src/interfaces/libpq/fe-secure.c 14 Jun 2003 17:49:54 -0000 1.24
> > --- src/interfaces/libpq/fe-secure.c 14 Jun 2003 18:08:54 -0000
> > ***************
> > *** 453,460 ****
> > if (addr.sa_family == AF_UNIX)
> > return 0;
> >
> > /* what do we know about the peer's common name? */
> > ! if ((h = gethostbyname(conn->peer_cn)) == NULL)
> > {
> > printfPQExpBuffer(&conn->errorMessage,
> > libpq_gettext("could not get information about host (%s): %s\n"),
>
> That code is inside #ifdef NOT_USED, which is why I didn't change
> it in my ipv6 patch.

Yes, I noticed that, but it seemed reasonable to get things consistent.

>
> Should I convert that function to use getaddrinfo too?
>
> Others I didn't change where pg_krb4_sendauth() and
> pg_krb5_sendauth(), because I don't have a clue about kerberos.

I understand. I guess we should be consisent if we can be.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Matthew Kirkwood 2003-06-14 19:32:40 Re: Pre-allocation of shared memory ...
Previous Message Kurt Roeckx 2003-06-14 18:44:43 Re: [HACKERS] PostgreSQL libraries - PThread Support, but

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2003-06-14 19:21:20 Re: Add thread.c and linkage
Previous Message Kurt Roeckx 2003-06-14 18:44:43 Re: [HACKERS] PostgreSQL libraries - PThread Support, but