On Thu, Mar 05, 2026 at 02:01:32AM +0100, Tomas Vondra wrote:
> Except that heap_freetuple is defined like this:
>
> void
> heap_freetuple(HeapTuple htup)
> {
> pfree(htup);
> }
>
> so this does not change anything. This report seems to miss how tuples
> are allocated in heap_form_tuple.Right, what is obviously an LLM analysis forgot the context of how HeapTuple allocations happen when these are formed. Saying that, this is new code, that I had the idea to commit like this, and I don't like fresh inconsistencies. We also use heap_freetuple() in upsert_pg_statistic_ext_data(). Hence, at the end, done. It was harmless as-is, still.. -- Michael
signature.asc
Description: PGP signature
