diff options
author | Alexander Korotkov <akorotkov@postgresql.org> | 2020-06-20 17:34:51 +0300 |
---|---|---|
committer | Alexander Korotkov <akorotkov@postgresql.org> | 2020-06-20 17:35:55 +0300 |
commit | 396a44a0153657bd4f5385b184b66b7ada8682ca (patch) | |
tree | 7e755a893016661a8059e964c8c9b8b4373c746d /src/backend/access/spgist/spgxlog.c | |
parent | 5b52008a64414cd161db11f4c0d52fc0d2ee304a (diff) | |
download | postgresql-396a44a0153657bd4f5385b184b66b7ada8682ca.tar.gz postgresql-396a44a0153657bd4f5385b184b66b7ada8682ca.zip |
Fix masking of SP-GiST pages during xlog consistency check
spg_mask() didn't take into account that pd_lower equal to SizeOfPageHeaderData
is still valid value. This commit fixes that. Backpatch to 11, where
spg_mask() pg_lower check was introduced.
Reported-by: Michael Paquier
Discussion: https://postgr.es/m/20200615131405.GM52676%40paquier.xyz
Backpatch-through: 11
Diffstat (limited to 'src/backend/access/spgist/spgxlog.c')
-rw-r--r-- | src/backend/access/spgist/spgxlog.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/access/spgist/spgxlog.c b/src/backend/access/spgist/spgxlog.c index ebe6ae8715b..6bebaaf0895 100644 --- a/src/backend/access/spgist/spgxlog.c +++ b/src/backend/access/spgist/spgxlog.c @@ -1008,6 +1008,6 @@ spg_mask(char *pagedata, BlockNumber blkno) * Mask the unused space, but only if the page's pd_lower appears to have * been set correctly. */ - if (pagehdr->pd_lower > SizeOfPageHeaderData) + if (pagehdr->pd_lower >= SizeOfPageHeaderData) mask_unused_space(page); } |