From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Joe Conway <mail(at)joeconway(dot)com> |
Cc: | PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] 7.4.2 Build broken on (Sparc) Solaris 7 and |
Date: | 2004-03-23 02:06:07 |
Message-ID: | 200403230206.i2N267n13448@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Joe Conway wrote:
> Bruce Momjian wrote:
> > OK, patch attached and applied. It adds configure tests for the 5-arg
> > version of getpwuid_r() and properly falls back to the Posix draft
> > version you have on Solaris. Seems Solaris 9 also still has the draft
> > version.
>
> > ! result = getpwuid_r(uid, resultbuf, buffer, buflen);
>
> Bruce,
>
> I'm getting core dumps on:
>
> 7.4.2 with this patch
> bash-2.03$ uname -a
> SunOS col65 5.8 Generic_108528-27 sun4u sparc SUNW,Sun-Fire-280R
>
> Shouldn't the line above actually be:
>
> *result = getpwuid_r(uid, resultbuf, buffer, buflen);
>
> The manpage shows this:
> struct passwd *getpwuid_r(uid_t uid, struct passwd *pwd,
> char *buffer, int buflen);
>
> and the function's parameter is this:
> struct passwd **result
>
> With that change, all seems well for me.
Yes, CVS HEAD and 7.4.X patched. Thanks.
--
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
From | Date | Subject | |
---|---|---|---|
Next Message | Neil Conway | 2004-03-23 02:56:50 | charset / postmaster doc improv |
Previous Message | Tom Lane | 2004-03-23 01:42:56 | Re: Initial eventlog support on win32 |