From 25b3ddd1de54563b631484f6d56bd56bef906102 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