aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>2015-07-27 12:30:26 +0300
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>2015-07-27 12:30:26 +0300
commit334445179c82ba304480ecbd00ae4237587e4baf (patch)
tree3901ee1914c6af9cb5e4188e1f2b714ff3f7eee1 /src
parent023430abf72eb7d335430e241065d5ed19ddd94b (diff)
downloadpostgresql-334445179c82ba304480ecbd00ae4237587e4baf.tar.gz
postgresql-334445179c82ba304480ecbd00ae4237587e4baf.zip
Reuse all-zero pages in GIN.
In GIN, an all-zeros page would be leaked forever, and never reused. Just add them to the FSM in vacuum, and they will be reinitialized when grabbed from the FSM. On master and 9.5, attempting to access the page's opaque struct also caused an assertion failure, although that was otherwise harmless. Reported by Jeff Janes. Backpatch to all supported versions.
Diffstat (limited to 'src')
-rw-r--r--src/backend/access/gin/ginvacuum.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/access/gin/ginvacuum.c b/src/backend/access/gin/ginvacuum.c
index eba572b0d8a..1315762ecf9 100644
--- a/src/backend/access/gin/ginvacuum.c
+++ b/src/backend/access/gin/ginvacuum.c
@@ -710,7 +710,7 @@ ginvacuumcleanup(PG_FUNCTION_ARGS)
LockBuffer(buffer, GIN_SHARE);
page = (Page) BufferGetPage(buffer);
- if (GinPageIsDeleted(page))
+ if (PageIsNew(page) || GinPageIsDeleted(page))
{
Assert(blkno != GIN_ROOT_BLKNO);
RecordFreeIndexPage(index, blkno);