Re: Solaris testers wanted for strxfrm() behavior

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Geoghegan <pg(at)heroku(dot)com>, Noah Misch <noah(at)leadboat(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Solaris testers wanted for strxfrm() behavior
Date: 2015-06-28 23:35:12
Message-ID: CA+TgmoYpVg1kbgPv-LNNABZeJSJAWthGgxzLS5nM1nKXOR4WoQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Jun 28, 2015 at 7:14 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Peter Geoghegan <pg(at)heroku(dot)com> writes:
>> It might have been the right decision at the time to paper over the
>> problem, but only for a year or two. I'd only favor adding defenses if
>> it could be expected to take longer for the Solaris stdlib people to
>> ship a fix for their egregious bug than it would take for the next
>> Postgres point release. Why should that be true, though?
>
>> This just happened to be one of the more embarrassing, obvious stdlib
>> bugs that appeared in the last 15 years, and so we're talking about it
>> now, but there are plenty more, a good deal of which are far more
>> recent than this one. Where does it end? I don't see why this bug is
>> special just because 4 or 5 people complained about it on
>> pgsql-hackers over a decade ago.
>
> Peter, your arguments are beginning to sound like desperate excuse-making.
>
> The reason that bug is "special" is that it looks like a crash in
> Postgres, one that people have complained of because they didn't see it
> in other programs, which is not totally surprising because it requires
> a somewhat unusual usage of strxfrm(). I think the dumb two-call
> implementation exhibited in convert_string_datum() is mainstream usage,
> which would explain why Sun hadn't noticed the bug ages ago.
>
> It might be all right to refuse to support platforms that have this
> bug, but I think we need to determine with some more clarity just
> how widespread the bug still is, and in any case we should at least
> install some defense that would allow us to report that libc is
> buggy. Otherwise we'll be back to fielding bug reports that trace
> to this, which is no fun for anyone and does not make us look good.

I completely agree. Noah is quite right to try to find out whether
this is still an issue, and I'm glad he's doing it, and I think it's
very unfortunate that Peter is trying to discourage that research. If
in fact the bug does not still exist in the wild, then Noah's research
will demonstrate that we have no problem, and we do not need to do
anything. If it does, then we can decide what to do about that. But
I have come to value Noah's diligent attitude towards hunting down
problems in our code base, and I hope Peter (and everyone else) will
appreciate that attitude as well - or at the very least, stay out of
the way.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Geoghegan 2015-06-28 23:35:17 Re: Solaris testers wanted for strxfrm() behavior
Previous Message Tom Lane 2015-06-28 23:27:17 Re: anole: assorted stability problems