diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-11-26 11:14:27 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2021-11-26 11:14:27 -0300 |
| commit | d24dac9549067716f5b4d8e44ccf87dd5274bfcd (patch) | |
| tree | fbf43883d373d4f38fc0406b1c037e9ef9b07e8e /src/backend/access/transam/xlogreader.c | |
| parent | 371087d006e04991080bf17cf2287db38d3ea92e (diff) | |
| download | postgresql-d24dac9549067716f5b4d8e44ccf87dd5274bfcd.tar.gz postgresql-d24dac9549067716f5b4d8e44ccf87dd5274bfcd.zip | |
Fix determination of broken LSN in OVERWRITTEN_CONTRECORD
In commit ff9f111bce24 I mixed up inconsistent definitions of the LSN of
the first record in a page, when the previous record ends exactly at the
page boundary. The correct LSN is adjusted to skip the WAL page header;
I failed to use that when setting XLogReaderState->overwrittenRecPtr,
so at WAL replay time VerifyOverwriteContrecord would refuse to let
replay continue past that record.
Backpatch to 10. 9.6 also contains this bug, but it's no longer being
maintained.
Discussion: https://postgr.es/m/45597.1637694259@sss.pgh.pa.us
Diffstat (limited to 'src/backend/access/transam/xlogreader.c')
| -rw-r--r-- | src/backend/access/transam/xlogreader.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/access/transam/xlogreader.c b/src/backend/access/transam/xlogreader.c index f01aea6ddad..d797d9d5087 100644 --- a/src/backend/access/transam/xlogreader.c +++ b/src/backend/access/transam/xlogreader.c @@ -464,7 +464,7 @@ restart: */ if (pageHeader->xlp_info & XLP_FIRST_IS_OVERWRITE_CONTRECORD) { - state->overwrittenRecPtr = state->currRecPtr; + state->overwrittenRecPtr = RecPtr; ResetDecoder(state); RecPtr = targetPagePtr; goto restart; |