RE: Thread-unsafe coding in ecpg

Поиск
Список
Период
Сортировка
От Tsunakawa, Takayuki
Тема RE: Thread-unsafe coding in ecpg
Дата
Msg-id 0A3221C70F24FB45833433255569204D1FB6D21E@G01JPEXMBYT05
обсуждение исходный текст
Ответ на Re: Thread-unsafe coding in ecpg  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
From: Tom Lane [mailto:tgl@sss.pgh.pa.us]
> BTW, I found another spot in descriptor.c where ecpglib is using
> setlocale() for the same purpose.  Perhaps that one's not reachable
> in threaded apps, but I didn't see any obvious reason to think so,
> so I changed it too.

Ouch, thanks.  And I'm sorry to annoy you by pointing out a trivial thing: in v3 patch, _configthreadlocale() is not
calledto restore the original value when setlocale() or ecpg_strdup() fails.  I hope this is fixed in v4.
 

+ #ifdef WIN32
+     stmt->oldthreadlocale = _configthreadlocale(_ENABLE_PER_THREAD_LOCALE);
+     if (stmt->oldthreadlocale == -1)
+     {
+         ecpg_do_epilogue(stmt);
+         return false;
+     }
+ #endif
      stmt->oldlocale = ecpg_strdup(setlocale(LC_NUMERIC, NULL), lineno);
      if (stmt->oldlocale == NULL)
      {

Regards
Takayuki Tsunakawa




В списке pgsql-hackers по дате отправления:

Предыдущее
От: Kyotaro HORIGUCHI
Дата:
Сообщение: Re: shared-memory based stats collector
Следующее
От: "Imai, Yoshikazu"
Дата:
Сообщение: RE: speeding up planning with partitions