From 5cd6538345858aec128945b58265a8d5560777dc Mon Sep 17 00:00:00 2001 From: Alvaro Herrera Date: Fri, 11 Sep 2015 15:29:46 -0300 Subject: [PATCH] Add missing ReleaseBuffer call in BRIN revmap code I think this particular branch is actually dead, but the analysis to prove that is not trivial, so instead take the weasel way. Reported by Jinyu Zhang Backpatch to 9.5, where BRIN was introduced. --- src/backend/access/brin/brin_revmap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/backend/access/brin/brin_revmap.c b/src/backend/access/brin/brin_revmap.c index 8c55f16f838..6ddcfda0a6c 100644 --- a/src/backend/access/brin/brin_revmap.c +++ b/src/backend/access/brin/brin_revmap.c @@ -432,6 +432,7 @@ revmap_physical_extend(BrinRevmap *revmap) if (needLock) UnlockRelationForExtension(irel, ExclusiveLock); LockBuffer(revmap->rm_metaBuf, BUFFER_LOCK_UNLOCK); + ReleaseBuffer(buf); return; } LockBuffer(buf, BUFFER_LOCK_EXCLUSIVE); -- 2.39.5