mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix race condition in gettext() initialization in libpq and ecpglib.

Browse Source 
In libpq and ecpglib, multiple threads can concurrently enter the
initialization logic for message localization.  Since we set the
its-done flag before actually doing the work, it'd be possible
for some threads to reach gettext() before anyone has called
bindtextdomain().  Barring bugs in libintl itself, this would not
result in anything worse than failure to localize some early
messages.  Nonetheless, it's a bug, and an easy one to fix.

Noted while investigating bug #17299 from Clemens Zeidler
(much thanks to Liam Bowen for followup investigation on that).
It currently appears that that actually *is* a bug in libintl itself,
but that doesn't let us off the hook for this bit.

Back-patch to all supported versions.

Discussion: https://postgr.es/m/17299-7270741958c0b1ab@postgresql.org
Discussion: https://postgr.es/m/CAE7q7Eit4Eq2=bxce=Fm8HAStECjaXUE=WBQc-sDDcgJQ7s7eg@mail.gmail.com
This commit is contained in:
Tom Lane 2022-01-21 15:36:12 -05:00
parent fd48e5f5d3
commit 64ebb43df0
2 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/interfaces/ecpg/ecpglib/misc.c
View File

@ -491,7 +491,14 @@ win32_pthread_once(volatile pthread_once_t *once, void (*fn) (void))
char *
ecpg_gettext(const char *msgid)
{
static bool already_bound = false;
/*
* If multiple threads come through here at about the same time, it's okay
* for more than one of them to call bindtextdomain(). But it's not okay
* for any of them to reach dgettext() before bindtextdomain() is
* complete, so don't set the flag till that's done. Use "volatile" just
* to be sure the compiler doesn't try to get cute.
*/
static volatile bool already_bound = false;
if (!already_bound)
{
@ -503,12 +510,12 @@ ecpg_gettext(const char *msgid)
#endif
const char *ldir;
already_bound = true;
/* No relocatable lookup here because the binary could be anywhere */
ldir = getenv("PGLOCALEDIR");
if (!ldir)
ldir = LOCALEDIR;
bindtextdomain(PG_TEXTDOMAIN("ecpglib"), ldir);
already_bound = true;
#ifdef WIN32
SetLastError(save_errno);
#else

11
src/interfaces/libpq/fe-misc.c
View File

@ -1271,7 +1271,14 @@ PQenv2encoding(void)
static void
libpq_binddomain(void)
{
static bool already_bound = false;
/*
* If multiple threads come through here at about the same time, it's okay
* for more than one of them to call bindtextdomain(). But it's not okay
* for any of them to return to caller before bindtextdomain() is
* complete, so don't set the flag till that's done. Use "volatile" just
* to be sure the compiler doesn't try to get cute.
*/
static volatile bool already_bound = false;
if (!already_bound)
{
@ -1283,12 +1290,12 @@ libpq_binddomain(void)
#endif
const char *ldir;
already_bound = true;
/* No relocatable lookup here because the binary could be anywhere */
ldir = getenv("PGLOCALEDIR");
if (!ldir)
ldir = LOCALEDIR;
bindtextdomain(PG_TEXTDOMAIN("libpq"), ldir);
already_bound = true;
#ifdef WIN32
SetLastError(save_errno);
#else