diff options
| author | Alexander Korotkov <akorotkov@postgresql.org> | 2020-06-20 17:34:51 +0300 |
|---|---|---|
| committer | Alexander Korotkov <akorotkov@postgresql.org> | 2020-06-20 18:14:51 +0300 |
| commit | 39aafc88c4b4ac281df8b2c2b8be72d4e4d99e9f (patch) | |
| tree | 4e9fb519e2058af184f6c148fe5c8a504c6d0585 | |
| parent | e6c6f427e356e3706ce2f0ae7e7e94e5501bbc13 (diff) | |
| download | postgresql-39aafc88c4b4ac281df8b2c2b8be72d4e4d99e9f.tar.gz postgresql-39aafc88c4b4ac281df8b2c2b8be72d4e4d99e9f.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 7be2291d07e..999d0ca15d5 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); } |