psycopg/psycopg2
1
1
Fork 0
mirror of https://github.com/psycopg/psycopg2.git synced 2025-02-07 21:00:33 +03:00
Code Issues Packages Projects Releases Wiki Activity

Don't call CLEARPGRES on the cursor state without holding the gil

Browse Source 
There is a chance it is executed by two different threads resulting in
issue #384.

I havent't found any other case that may lead to double free.
This commit is contained in:
Daniele Varrazzo 2019-01-02 13:15:00 +01:00
parent ddcf808d56
commit 07d9fb8718
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
psycopg/pqpath.c
View File

@ -1016,6 +1016,8 @@ pq_execute(cursorObject *curs, const char *query, int async, int no_result, int
}
Dprintf("pq_execute: pg connection at %p OK", curs->conn->pgconn);
CLEARPGRES(curs->pgres);
Py_BEGIN_ALLOW_THREADS;
pthread_mutex_lock(&(curs->conn->lock));
@ -1027,7 +1029,6 @@ pq_execute(cursorObject *curs, const char *query, int async, int no_result, int
}
if (async == 0) {
CLEARPGRES(curs->pgres);
Dprintf("pq_execute: executing SYNC query: pgconn = %p", curs->conn->pgconn);
Dprintf(" %-.200s", query);
if (!psyco_green()) {
@ -1069,7 +1070,6 @@ pq_execute(cursorObject *curs, const char *query, int async, int no_result, int
Dprintf("pq_execute: executing ASYNC query: pgconn = %p", curs->conn->pgconn);
Dprintf(" %-.200s", query);
CLEARPGRES(curs->pgres);
if (PQsendQuery(curs->conn->pgconn, query) == 0) {
if (CONNECTION_BAD == PQstatus(curs->conn->pgconn)) {
curs->conn->closed = 2;