diff options
| author | Fujii Masao <fujii@postgresql.org> | 2019-11-07 16:31:36 +0900 |
|---|---|---|
| committer | Fujii Masao <fujii@postgresql.org> | 2019-11-07 16:32:37 +0900 |
| commit | e5cfb8cbbe91e73ee92d9e4ab023ca208f3b748a (patch) | |
| tree | 15251333fcda436c6df48e347dc65aec35162268 /src | |
| parent | f96f14744533e7409876056eeca0229a32fac4dd (diff) | |
| download | postgresql-e5cfb8cbbe91e73ee92d9e4ab023ca208f3b748a.tar.gz postgresql-e5cfb8cbbe91e73ee92d9e4ab023ca208f3b748a.zip | |
Fix assertion failure when running pgbench -s.
If there is the WAL page that the continuation WAL record just fits within
(i.e., the continuation record ends just at the end of the page) and
the LSN in such page is specified with -s option, previously pg_waldump
caused an assertion failure. The cause of this assertion failure was that
XLogFindNextRecord() that pg_waldump -s calls mistakenly handled
such special WAL page.
This commit changes XLogFindNextRecord() so that it can handle
such WAL page correctly.
Back-patch to all supported versions.
Author: Andrey Lepikhov
Reviewed-by: Fujii Masao, Michael Paquier
Discussion: https://postgr.es/m/99303554-5dd5-06e6-f943-b3005ccd6edd@postgrespro.ru
Diffstat (limited to 'src')
| -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 88be7fe0223..7d40029b083 100644 --- a/src/backend/access/transam/xlogreader.c +++ b/src/backend/access/transam/xlogreader.c @@ -955,7 +955,7 @@ XLogFindNextRecord(XLogReaderState *state, XLogRecPtr RecPtr) * * Note that record headers are MAXALIGN'ed */ - if (MAXALIGN(header->xlp_rem_len) > (XLOG_BLCKSZ - pageHeaderSize)) + if (MAXALIGN(header->xlp_rem_len) >= (XLOG_BLCKSZ - pageHeaderSize)) tmpRecPtr = targetPagePtr + XLOG_BLCKSZ; else { |