pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Daniel Gustafsson <dgustafsson@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Exclude Threadsanitizer instrumentation in exit check
Date: Wed, 31 Jan 2024 22:02:54 +0000
Message-ID: <E1rVIfW-0043Wx-UB@gemulon.postgresql.org> (raw)

Exclude Threadsanitizer instrumentation in exit check

When building libpq there is a check to ensure that we're not
linking against code that calls exit(). This check is using a
heuristic grep with exclusions for known false positives. The
Threadsanitizer library instrumentation for function exits is
named such that it triggers the check, so add an exclusion.

This fix is only applied to the Makefile since the meson build
files don't yet have this check.  Adding the check to meson is
outside the scope of this patch though.

Reported-by: Roman Lozko <lozko.roma@gmail.com>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/CAEhC_BmNGKgj2wKArH2EAU11BsaHYgLnrRFJGRm5Vs8WJzyiQA@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/936f56988741f17cce8731cfa6150fad96e571e9

Modified Files
--------------
src/interfaces/libpq/Makefile | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)



Message-ID: <E1rVIfW-0043Wx-UB@gemulon.postgresql.org>
Permalink:  ../E1rVIfW-0043Wx-UB@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1rVIfW-0043Wx-UB@gemulon.postgresql.org

 · 

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-committers@postgresql.org
  Cc: dgustafsson@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Exclude Threadsanitizer instrumentation in exit check
  In-Reply-To: <E1rVIfW-0043Wx-UB@gemulon.postgresql.org>

* 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