From: Tom Lane <tgl@sss.pgh.pa.us>
To: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: pgsql: Refactor dlopen() support
Date: Fri, 07 Sep 2018 10:19:06 -0400
Message-ID: <3574.1536329946@sss.pgh.pa.us> (raw)
In-Reply-To: <765f9eb2-789c-59b2-c4c3-8ee865038b58@2ndquadrant.com>
References: <E1fxqjl-0003YS-NS@gemulon.postgresql.org>
<14508.1536301857@sss.pgh.pa.us>
<765f9eb2-789c-59b2-c4c3-8ee865038b58@2ndquadrant.com>
Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
> On 07/09/2018 08:30, Tom Lane wrote:
>> Buildfarm member locust doesn't like this much. I've been able to
>> reproduce the problem on an old Mac laptop running the same macOS release,
>> viz 10.5.8. (Note that we're not seeing it on earlier or later releases,
>> which is odd in itself.)
> Nothing should have changed on macOS except that the intermediate
> functions pg_dl*() were replaced by direct calls to dl*(). Very strange.
Somehow or other, the changes you made in dfmgr.c's #include lines
have made it so that find_rendezvous_variable's local "bool found"
variable is actually of type _Bool (which is word-wide on these
machines). However, hash_search thinks its output variable is
of type pointer to "typedef char bool". The proximate cause of
the observed failure is that find_rendezvous_variable sees "found"
as true when it should not, and thus fails to zero out the variable's
value.
No time to look further right now, but there's something rotten
about the way we're handling bool.
regards, tom lane
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-committers@postgresql.org
Cc: tgl@sss.pgh.pa.us, peter.eisentraut@2ndquadrant.com, pgsql-hackers@lists.postgresql.org
Subject: Re: pgsql: Refactor dlopen() support
In-Reply-To: <3574.1536329946@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox