Re: Fix incorrect assertion in heapgettup_pagemode()

On 25/12/2025 15:34, Chao Li wrote:
>> On Dec 25, 2025, at 19:13, cca5507 <cca5507(at)qq(dot)com> wrote:
>> diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c
>> index 469397e7344..1229e22e78a 100644
>> --- a/src/backend/access/heap/heapam.c
>> +++ b/src/backend/access/heap/heapam.c
>> @@ -1077,7 +1077,7 @@ continue_page:
>> ItemId lpp;
>> OffsetNumber lineoff;
>>
>> - Assert(lineindex <= scan->rs_ntuples);
>> + Assert(lineindex < scan->rs_ntuples);
>> lineoff = scan->rs_vistuples[lineindex];
>> lpp = PageGetItemId(page, lineoff);
>> Assert(ItemIdIsNormal(lpp));
>>
>> The lineindex is 0-based, so it should be '<' rather than '<=', thoughts?
>>
>> --
>> Regards,
>> ChangAo Chen
>> <v1-0001-Fix-incorrect-assertion-in-heapgettup_pagemode.patch>
>
> Good catch. The function comment clearly mentions that lineindex is 0 based.
> ...
> So, the fix looks correct.

Committed, thanks!

- Heikki