From 16e7bcfac57fc21dd2124b39132c58971e41448a Mon Sep 17 00:00:00 2001 From: Amit Kapila Date: Tue, 4 Sep 2018 08:33:33 +0530 Subject: [PATCH] During the split, set checksum on an empty hash index page. On a split, we allocate a new splitpoint's worth of bucket pages wherein we initialize the last page with zeros which is fine, but we forgot to set the checksum for that last page. We decided to back-patch this fix till 10 because we don't have an easy way to test it in prior versions. Another reason is that the hash-index code is changed heavily in 10, so it is not advisable to push the fix without testing it in prior versions. Author: Amit Kapila Reviewed-by: Yugo Nagata Backpatch-through: 10 Discussion: https://postgr.es/m/5d03686d-727c-dbf8-0064-bf8b97ffe850@2ndquadrant.com --- src/backend/access/hash/hashpage.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/backend/access/hash/hashpage.c b/src/backend/access/hash/hashpage.c index 69a9c4f1fee..6825c14309b 100644 --- a/src/backend/access/hash/hashpage.c +++ b/src/backend/access/hash/hashpage.c @@ -1038,6 +1038,7 @@ _hash_alloc_buckets(Relation rel, BlockNumber firstblock, uint32 nblocks) true); RelationOpenSmgr(rel); + PageSetChecksumInplace(page, lastblock); smgrextend(rel->rd_smgr, MAIN_FORKNUM, lastblock, zerobuf.data, false); return true;