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:59 +0300 |
commit | 3f88e2d78dbe61b8eec8dfb051b9dc0d1ff0052f (patch) | |
tree | e6922325cd7196b64e59c041c26ca14dc46d59a3 | |
parent | 2e155d90d1b867f7b5bf9616236386eb1e0f40ff (diff) | |
download | postgresql-3f88e2d78dbe61b8eec8dfb051b9dc0d1ff0052f.tar.gz postgresql-3f88e2d78dbe61b8eec8dfb051b9dc0d1ff0052f.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
-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 9e2bd3f8119..314f6eaa81f 100644 --- a/src/backend/access/spgist/spgxlog.c +++ b/src/backend/access/spgist/spgxlog.c @@ -1043,6 +1043,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); } |