Re: BUG #15367: Crash in pg_fe_scram_free when using foreign tables

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk>
Cc: Jeremy Evans <code(at)jeremyevans(dot)net>, Michael Paquier <michael(at)paquier(dot)xyz>, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #15367: Crash in pg_fe_scram_free when using foreign tables
Date: 2018-09-10 16:26:04
Message-ID: 16984.1536596764@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

I wrote:
> AFAICT, this platform's "ld" doesn't have any simple equivalent of
> a version script, but it does have "-B symbolic", which appears to
> fix the problem in a quick test.

BTW, I discovered while rummaging through the git history that we
used to use -Bsymbolic on most platforms --- but only for ODBC,
which had exactly the problem we're fighting now of an intended-to-
be-local reference being resolved outside the library, but in a place
where the malfunction was very obvious. When we removed ODBC from the
core distro, the Makefile support for -Bsymbolic eventually went away
too. But back around 7.2 we had this:

$ grep -r shlib_symbolic /home/postgres/REL7_2/pgsql
/home/postgres/REL7_2/pgsql/src/interfaces/odbc/GNUmakefile: 54: LINK.shared += $(shlib_symbolic)
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.bsdi: 19: shlib_symbolic = -Wl,-Bsymbolic
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.freebsd: 6: shlib_symbolic = -Wl,-Bsymbolic -lc
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.hpux: 33: shlib_symbolic = -Bsymbolic
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.irix5: 4: shlib_symbolic = -Wl,-B,symbolic
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.linux: 4: shlib_symbolic = -Wl,-Bsymbolic
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.netbsd: 6: shlib_symbolic = -Wl,-Bsymbolic -lc
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.openbsd: 6: shlib_symbolic = -Wl,-Bsymbolic
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.sco: 3: shlib_symbolic = -Wl,-Bsymbolic
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.solaris: 11: shlib_symbolic = -Wl,-Bsymbolic
/home/postgres/REL7_2/pgsql/src/makefiles/Makefile.unixware: 6: shlib_symbolic = -Wl,-Bsymbolic

This suggests that we're gonna need -Bsymbolic on Solaris too,
which agrees with your remark upthread. Unfortunately, with
all the Solaris buildfarm members being hors de C99 right now,
there's no easy way to test. I'm inclined to go fix the problem
that's blocking castoroides, at least.

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Bruce Momjian 2018-09-10 16:47:35 Re: BUG #15376: Postgres sql 9.4.19 pg_upgrade stops with error The source cluster was not shut down cleanly.
Previous Message Bruce Momjian 2018-09-10 16:14:43 Re: BUG #15376: Postgres sql 9.4.19 pg_upgrade stops with error The source cluster was not shut down cleanly.