pgsql: Fix ancient bug in ecpg's pthread_once() emulation for Windows. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix ancient bug in ecpg's pthread_once() emulation for Windows.
Date
Msg-id E1kWN6k-0002Ns-CC@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix ancient bug in ecpg's pthread_once() emulation for Windows.

We must not set the "done" flag until after we've executed the
initialization function.  Otherwise, other threads can fall through
the initial unlocked test before initialization is really complete.

This has been seen to cause rare failures of ecpg's thread/descriptor
test, and it could presumably cause other sorts of misbehavior in
threaded ECPG-using applications, since ecpglib relies on
pthread_once() in several places.

Diagnosis and patch by me, based on investigation by Alexander Lakhin.
Back-patch to all supported branches (the bug dates to 2007).

Discussion: https://postgr.es/m/16685-d6cd241872c101d3@postgresql.org

Branch
------
REL9_5_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/758b45e83b50b08a0bca03a9b5610a27ed7c7b09

Modified Files
--------------
src/interfaces/ecpg/ecpglib/misc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Fix issue with --enable-coverage and the new unicode {de,re}comp
Next
From: David Rowley
Date:
Subject: pgsql: Fix incorrect parameter name in a function header comment