diff options
author | Michael Paquier <michael@paquier.xyz> | 2023-03-02 14:03:08 +0900 |
---|---|---|
committer | Michael Paquier <michael@paquier.xyz> | 2023-03-02 14:03:08 +0900 |
commit | 9d41ecfcd9a7cb4ec6b20add4a55603ebba03f0d (patch) | |
tree | f432b94d5dfb9577e59c19d34518e7c2641d020f /src/backend/executor/nodeModifyTable.c | |
parent | eae09137d53ecb9cb4c1ba7624723f1c1cbebeec (diff) | |
download | postgresql-9d41ecfcd9a7cb4ec6b20add4a55603ebba03f0d.tar.gz postgresql-9d41ecfcd9a7cb4ec6b20add4a55603ebba03f0d.zip |
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
Diffstat (limited to 'src/backend/executor/nodeModifyTable.c')
0 files changed, 0 insertions, 0 deletions