postgres.git / summary / log / commit / refs

commit    25b3ddd1de54563b631484f6d56bd56bef906102
Author:   Alvaro Herrera <alvherre@alvh.no-ip.org>
Date:     Fri Sep 11 18:29:46 2015 +0000
First released in: 9.6.0, 9.5.0

    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);


[parent: 974f910b6451]