pgsql: Fix RBM_ZERO_AND_LOCK mode to not acquire lock on local buffers. - Mailing list pgsql-committers

From Heikki Linnakangas
Subject pgsql: Fix RBM_ZERO_AND_LOCK mode to not acquire lock on local buffers.
Date
Msg-id E1YsQuR-0001Bo-BO@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix RBM_ZERO_AND_LOCK mode to not acquire lock on local buffers.

Commit 81c45081 introduced a new RBM_ZERO_AND_LOCK mode to ReadBuffer, which
takes a lock on the buffer before zeroing it. However, you cannot take a
lock on a local buffer, and you got a segfault instead. The version of that
patch committed to master included a check for !isLocalBuf, and therefore
didn't crash, but oddly I missed that in the back-patched versions. This
patch adds that check to the back-branches too.

RBM_ZERO_AND_LOCK mode is only used during WAL replay, and in hash indexes.
WAL replay only deals with shared buffers, so the only way to trigger the
bug is with a temporary hash index.

Reported by Artem Ignatyev, analysis by Tom Lane.

Branch
------
REL9_3_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/96b676cc66c5a60a522364487bf7c7a9593bb229

Modified Files
--------------
src/backend/storage/buffer/bufmgr.c |    3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: pgsql: Add pgstattuple_approx() to the pgstattuple extension.
Next
From: Heikki Linnakangas
Date:
Subject: pgsql: Fix RBM_ZERO_AND_LOCK mode to not acquire lock on local buffers.