pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrew Gierth <andrew@tao11.riddles.org.uk>
Cc: Jeremy Evans <code@jeremyevans.net>
Cc: Michael Paquier <michael@paquier.xyz>
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #15367: Crash in pg_fe_scram_free when using foreign tables
Date: Mon, 10 Sep 2018 10:52:46 -0400
Message-ID: <12293.1536591166@sss.pgh.pa.us> (raw)
In-Reply-To: <12134.1536550702@sss.pgh.pa.us>
References: <153626613985.23143.4743626885618266803@wrigleys.postgresql.org>
	<20180906215836.GP2726@paquier.xyz>
	<20180906222843.GF17425@jeremyevans.local>
	<20180907165103.GH17425@jeremyevans.local>
	<20180907175518.GI17425@jeremyevans.local>
	<20180907181522.GB32058@paquier.xyz>
	<20016.1536351535@sss.pgh.pa.us>
	<20180907204403.GC32058@paquier.xyz>
	<16558.1536407783@sss.pgh.pa.us>
	<18398.1536410055@sss.pgh.pa.us>
	<87o9d8rv08.fsf@news-spur.riddles.org.uk>
	<22025.1536521375@sss.pgh.pa.us>
	<87zhwqqwz7.fsf@news-spur.riddles.org.uk>
	<24041.1536523851@sss.pgh.pa.us>
	<12134.1536550702@sss.pgh.pa.us>

I wrote:
> Ugh ... anole just reported in with the same issue, which means it
> exists on HP-UX versions that people might still care about in
> practice.
> I still haven't looked for a fix (real life has intruded more than
> usual right at the moment).

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.  I'm running a more thorough test
now.

The fix I'm able to test looks like

 ifeq ($(PORTNAME), hpux)
   ifdef SO_MAJOR_VERSION
     shlib			= lib$(NAME)$(DLSUFFIX).$(SO_MAJOR_VERSION)
   endif
   ifeq ($(with_gnu_ld), yes)
     LINK.shared		= $(CC) -shared
     ifdef soname
       LINK.shared	+= -Wl,-h -Wl,$(soname)
     endif
   else
-    LINK.shared		= $(LD) -b
+    LINK.shared		= $(LD) -b -B symbolic
     ifdef soname
       LINK.shared	+= +h $(soname)
     endif

I'm not sure what to do in the with_gnu_ld subsection: building with
GNU ld might or might not have the problem, and if it does, the best
fix might or might not be "-B symbolic".  But none of the HPUX animals
in the buildfarm are using GNU ld, so I suspect that that subsection is
dead code.  I'm inclined to just add a comment saying that that code
has not been tested recently.

			regards, tom lane




view thread (33+ messages)  latest in thread

Message-ID: <12293.1536591166@sss.pgh.pa.us>
Permalink:  ../12293.1536591166@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/12293.1536591166@sss.pgh.pa.us

 · 

reply
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-bugs@postgresql.org
  Cc: tgl@sss.pgh.pa.us, andrew@tao11.riddles.org.uk, code@jeremyevans.net, michael@paquier.xyz, pgsql-bugs@lists.postgresql.org
  Subject: Re: BUG #15367: Crash in pg_fe_scram_free when using foreign tables
  In-Reply-To: <12293.1536591166@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