From 6a93535798aa9219d01c01c67161a78780e2af1a Mon Sep 17 00:00:00 2001 From: David Rowley Date: Tue, 29 Sep 2026 15:46:45 +1300 Subject: [PATCH] Fix incorrect calcSumX2 in numeric deserialize functions calcSumX2 is used to inform the aggregate transition function whether it should include the square of the transition value in the aggregate state. The deserialize functions for various numeric aggregates that do use calcSumX2 weren't properly setting the property to true. This does not matter in core as we never perform any aggregate transitions after a deserialize operation. This fix is only required for external code which plumbs aggregate paths together in such a way that could result in aggregate transitions being performed after states have been deserialized. Author: Maxime Schoemans Reviewed-by: Andrey Borodin Discussion: https://postgr.es/m/FEBF5C3D-CC53-455F-AA9D-4D377A2E957C@enterprisedb.com Backpatch-through: 14 --- src/backend/utils/adt/numeric.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/backend/utils/adt/numeric.c b/src/backend/utils/adt/numeric.c index 8b2575b63e3..b1be0beb080 100644 --- a/src/backend/utils/adt/numeric.c +++ b/src/backend/utils/adt/numeric.c @@ -5365,7 +5365,7 @@ numeric_deserialize(PG_FUNCTION_ARGS) initReadOnlyStringInfo(&buf, VARDATA_ANY(sstate), VARSIZE_ANY_EXHDR(sstate)); - result = makeNumericAggStateCurrentContext(false); + result = makeNumericAggStateCurrentContext(true); /* N */ result->N = pq_getmsgint64(&buf); @@ -5698,7 +5698,7 @@ numeric_poly_deserialize(PG_FUNCTION_ARGS) initReadOnlyStringInfo(&buf, VARDATA_ANY(sstate), VARSIZE_ANY_EXHDR(sstate)); - result = makeInt128AggStateCurrentContext(false); + result = makeInt128AggStateCurrentContext(true); /* N */ result->N = pq_getmsgint64(&buf);