]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix ancient bug in ecpg's pthread_once() emulation for Windows.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 24 Oct 2020 17:12:08 +0000 (13:12 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 24 Oct 2020 17:12:31 +0000 (13:12 -0400)
commit12a73f29ff7d00ea96ec4c5d1bef9acb5985d688
treeb7d98a8fa1aa2a836e0873a3846c30a871313e48
parent02f86d52382c3de5e3e506f6bd030a14886d2cff
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
src/interfaces/ecpg/ecpglib/misc.c