At update of non-LP_NORMAL TID, fail instead of corrupting page header.

Noah Misch <noah@leadboat.com>

Commit: 25e99483c4116313b678cc94df53d3ea28977752
Author: Noah Misch <noah@leadboat.com>
Date: 2025-01-25T19:28:19Z
Releases: 14.16
At update of non-LP_NORMAL TID, fail instead of corrupting page header.

The right mix of DDL and VACUUM could corrupt a catalog page header such
that PageIsVerified() durably fails, requiring a restore from backup.
This affects only catalogs that both have a syscache and have DDL code
that uses syscache tuples to construct updates.  One of the test
permutations shows a variant not yet fixed.

This makes !TransactionIdIsValid(TM_FailureData.xmax) possible with
TM_Deleted.  I think core and PGXN are indifferent to that.

Per bug #17821 from Alexander Lakhin.  Back-patch to v13 (all supported
versions).  The test case is v17+, since it uses INJECTION_POINT.

Discussion: https://postgr.es/m/17821-dd8c334263399284@postgresql.org

Files

PathChange+/−
src/backend/access/heap/heapam.c modified +44 −1
src/include/access/tableam.h modified +2 −1

Discussion