Hello.
At Thu, 14 Mar 2019 07:17:08 +0000, "Matsumura, Ryo" <matsumura.ryo@jp.fujitsu.com> wrote in
<03040DFF97E6E54E88D3BFEE5F5480F737AC390D@G01JPEXMBYT04>
> Hi
>
> I'm afraid that PREPARE of ecpglib is not thread safe.
> The following global variables are modified without any locking system.
> Is it unnecessary worry?
>
> [interfaces/ecpg/ecpglib/prepare.c]
> static int nextStmtID = 1;
> static stmtCacheEntry *stmtCacheEntries = NULL;
> static struct declared_statement *g_declared_list;
A connection cannot be concurrently used by multiple threads so
the programmer must guard connections using mutex [1] or
friends. If it is done by a single mutex (I suppose it is
common.), there's no race condition also on the prepared
statement storage. I'm not sure it is explicitly aimed but I
suppose that there's no problem in a common usage of the library.
[1] https://www.postgresql.org/docs/current/ecpg-connect.html
> If your application uses multiple threads of execution, they
> cannot share a connection concurrently. You must either
> explicitly control access to the connection (using mutexes) or
> use a connection for each thread.
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
