pageinspect: Fix crash with gist_page_items()
authorMichael Paquier
Thu, 2 Mar 2023 05:03:21 +0000 (14:03 +0900)
committerMichael Paquier
Thu, 2 Mar 2023 05:03:21 +0000 (14:03 +0900)
commit5ad63eee13e70eeff9659bcee024e8249b6bf68c
treed034ef97e7e2cc1882280db5ba36b8349994f056
parent1a9356f657e19ae1abeb0ffea0b7edaf69e315cb
pageinspect: Fix crash with gist_page_items()

Attempting to use this function with a raw page not coming from a GiST
index would cause a crash, as it was missing the same sanity checks as
gist_page_items_bytea().  This slightly refactors the code so as all the
basic validation checks for GiST pages are done in a single routine,
in the same fashion as the pageinspect functions for hash and BRIN.

This fixes an issue similar to 076f4d9.  A test is added to stress for
this case.  While on it, I have added a similar test for
brin_page_items() with a combination make of a valid GiST index and a
raw btree page.  This one was already protected, but it was not tested.

Reported-by: Egor Chindyaskin
Author: Dmitry Koval
Discussion: https://postgr.es/m/17815-fc4a2d3b74705703@postgresql.org
Backpatch-through: 14
contrib/pageinspect/expected/brin.out
contrib/pageinspect/expected/gist.out
contrib/pageinspect/gistfuncs.c
contrib/pageinspect/sql/brin.sql
contrib/pageinspect/sql/gist.sql