Skip site navigation (1) Skip section navigation (2)

pgsql: Fix Windows implementation of PGSemaphoreLock.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix Windows implementation of PGSemaphoreLock.
Date: 2012-05-10 17:36:58
Message-ID: E1SSXIQ-0004YA-VO@gemulon.postgresql.org (view raw or flat)
Thread:
Lists: pgsql-committers
Fix Windows implementation of PGSemaphoreLock.

The original coding failed to reset ImmediateInterruptOK before returning,
which would potentially allow a subsequent query-cancel interrupt to be
accepted at an unsafe point.  This is a really nasty bug since it's so hard
to predict the consequences, but they could be unpleasant.

Also, ensure that signal handlers are serviced before this function
returns, even if the semaphore is already set.  This should make the
behavior more like Unix.

Back-patch to all supported versions.

Branch
------
REL9_0_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/b149d1f90e7d42f719babc0c26addaeffa18df8c

Modified Files
--------------
src/backend/port/win32_sema.c |   20 ++++++++++++--------
1 files changed, 12 insertions(+), 8 deletions(-)

pgsql-committers by date

Next:From: Tom LaneDate: 2012-05-10 17:36:59
Subject: pgsql: Fix Windows implementation of PGSemaphoreLock.
Previous:From: Tom LaneDate: 2012-05-10 17:26:57
Subject: pgsql: Improve Windows implementation of WaitLatch/WaitLatchOrSocket.

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group