pgsql: Add hardening to catch invalid TIDs in indexes. - Mailing list pgsql-committers

From Peter Geoghegan
Subject pgsql: Add hardening to catch invalid TIDs in indexes.
Date
Msg-id E1mipN3-0003xz-TP@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Add hardening to catch invalid TIDs in indexes.

Add hardening to the heapam index tuple deletion path to catch TIDs in
index pages that point to a heap item that index tuples should never
point to.  The corruption we're trying to catch here is particularly
tricky to detect, since it typically involves "extra" (corrupt) index
tuples, as opposed to the absence of required index tuples in the index.

For example, a heap TID from an index page that turns out to point to an
LP_UNUSED item in the heap page has a good chance of being caught by one
of the new checks.  There is a decent chance that the recently fixed
parallel VACUUM bug (see commit 9bacec15) would have been caught had
that particular check been in place for Postgres 14.  No backpatch of
this extra hardening for now, though.

Author: Peter Geoghegan <pg@bowt.ie>
Reviewed-By: Andres Freund <andres@anarazel.de>
Discussion: https://postgr.es/m/CAH2-Wzk-4_raTzawWGaiqNvkpwDXxv3y1AQhQyUeHfkU=tFCeA@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/e7428a99a13f973549aab30c57ec8380ddda1869

Modified Files
--------------
src/backend/access/heap/heapam.c      | 65 +++++++++++++++++++++++++++++++++++
src/backend/access/index/genam.c      |  7 ++--
src/backend/access/nbtree/nbtdedup.c  |  2 ++
src/backend/access/nbtree/nbtinsert.c |  2 ++
src/include/access/tableam.h          |  2 ++
5 files changed, 76 insertions(+), 2 deletions(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Add support for LZ4 compression in pg_receivewal
Next
From: Michael Paquier
Date:
Subject: pgsql: Improve psql tab completion for COMMENT