From 18ea6b3d0da357b5874ab9cf41350d4473b01ee9 Mon Sep 17 00:00:00 2001 From: Peter Geoghegan Date: Mon, 18 Nov 2024 13:35:28 -0500 Subject: [PATCH] nbtree: consistently use minoff variable. This was arguably an oversight in commit 29b64d1de7, which moved this code from nbtutils.c to its nbtsearch.c caller. --- src/backend/access/nbtree/nbtsearch.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/backend/access/nbtree/nbtsearch.c b/src/backend/access/nbtree/nbtsearch.c index bf2dc9682ec..0cd04661366 100644 --- a/src/backend/access/nbtree/nbtsearch.c +++ b/src/backend/access/nbtree/nbtsearch.c @@ -1810,8 +1810,7 @@ _bt_readpage(IndexScanDesc scan, ScanDirection dir, OffsetNumber offnum, */ if (scan->ignore_killed_tuples && ItemIdIsDead(iid)) { - Assert(offnum >= P_FIRSTDATAKEY(opaque)); - if (offnum > P_FIRSTDATAKEY(opaque)) + if (offnum > minoff) { offnum = OffsetNumberPrev(offnum); continue; -- 2.30.2