summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Lane2023-04-17 18:22:06 +0000
committerTom Lane2023-04-17 18:22:26 +0000
commit3e383f9b68f13502789b234c2dc3274958bdbf78 (patch)
tree365ce41f95535dd4aeee835ed8c070048a94fb4e
parentcd7cdc550c2a00027d6ec1810d064ad636250b1f (diff)
Avoid trying to write an empty WAL record in log_newpage_range().
If the last few pages in the specified range are empty (all zero), then log_newpage_range() could try to emit an empty WAL record containing no FPIs. This at least upsets an Assert in ReserveXLogInsertLocation, and might perhaps have bad real-world consequences in non-assert builds. This has been broken since log_newpage_range() was introduced, but the case was hard if not impossible to hit before commit 3d6a98457 decided it was okay to leave VM and FSM pages intentionally zero. Nonetheless, it seems prudent to back-patch. log_newpage_range() was added in v12 but later back-patched, so this affects all supported branches. Matthias van de Meent, per report from Justin Pryzby Discussion: https://postgr.es/m/[email protected]
-rw-r--r--src/backend/access/transam/xloginsert.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/src/backend/access/transam/xloginsert.c b/src/backend/access/transam/xloginsert.c
index e3d5937479..ea7e2f67af 100644
--- a/src/backend/access/transam/xloginsert.c
+++ b/src/backend/access/transam/xloginsert.c
@@ -1304,6 +1304,10 @@ log_newpage_range(Relation rel, ForkNumber forknum,
blkno++;
}
+ /* Nothing more to do if all remaining blocks were empty. */
+ if (nbufs == 0)
+ break;
+
/* Write WAL record for this batch. */
XLogBeginInsert();