From d40abd5fcfb25d764419f8e9bffa5cdbdb247c1b Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Wed, 6 Nov 2019 14:20:29 +0100 Subject: [PATCH] Fix memory allocation mistake The previous code was allocating more memory than necessary because the formula used the wrong data type. Reported-by: Jehan-Guillaume de Rorthais Discussion: https://www.postgresql.org/message-id/20191105172918.3e32a446@firost --- src/backend/replication/logical/relation.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/backend/replication/logical/relation.c b/src/backend/replication/logical/relation.c index 85269c037d..f938d1fa48 100644 --- a/src/backend/replication/logical/relation.c +++ b/src/backend/replication/logical/relation.c @@ -267,7 +267,7 @@ logicalrep_rel_open(LogicalRepRelId remoteid, LOCKMODE lockmode) */ desc = RelationGetDescr(entry->localrel); oldctx = MemoryContextSwitchTo(LogicalRepRelMapContext); - entry->attrmap = palloc(desc->natts * sizeof(int)); + entry->attrmap = palloc(desc->natts * sizeof(AttrNumber)); MemoryContextSwitchTo(oldctx); found = 0;