From 310a8f6a700abc82f37934dc58808af25e077882 Mon Sep 17 00:00:00 2001 From: "Chao Li (Evan)" Date: Tue, 4 Nov 2025 16:45:40 +0800 Subject: [PATCH v1] Fix redundant zero-initialization in heap_form_minimal_tuple() heap_form_minimal_tuple() performed both palloc0() and memset(0), resulting in unnecessary double zeroing. This appears to have been introduced by commit a0942f4 as an oversight. While simply removing the memset() would also correct the issue, similar patterns elsewhere allocate with palloc() and then memset() only the required "extra" bytes. To keep the code consistent with those cases, replace palloc0() with palloc() here. Author: Chao Li --- src/backend/access/common/heaptuple.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/access/common/heaptuple.c b/src/backend/access/common/heaptuple.c index 1173a6d81b5..fff2f8802f8 100644 --- a/src/backend/access/common/heaptuple.c +++ b/src/backend/access/common/heaptuple.c @@ -1501,7 +1501,7 @@ heap_form_minimal_tuple(TupleDesc tupleDescriptor, /* * Allocate and zero the space needed. */ - mem = palloc0(len + extra); + mem = palloc(len + extra); memset(mem, 0, extra); tuple = (MinimalTuple) (mem + extra); -- 2.39.5 (Apple Git-154)