Fix memory leak due to LogicalRepRelMapEntry.attrmap.
authorAmit Kapila
Thu, 23 Jun 2022 03:50:41 +0000 (09:20 +0530)
committerAmit Kapila
Thu, 23 Jun 2022 03:50:41 +0000 (09:20 +0530)
When rebuilding the relation mapping on subscribers, we were not releasing
the attribute mapping's memory which was no longer required.

The attribute mapping used in logical tuple conversion was refactored in
PG13 (by commit e1551f96e6) but we forgot to update the related code that
frees the attribute map.

Author: Hou Zhijie
Reviewed-by: Amit Langote, Amit Kapila, Shi yu
Backpatch-through: 10, where it was introduced
Discussion: https://postgr.es/m/OSZPR01MB6310F46CD425A967E4AEF736FDA49@OSZPR01MB6310.jpnprd01.prod.outlook.com

src/backend/replication/logical/relation.c

index 6265c26b4aef3b607f0f74151acedc4dad8026bc..a5e5bf9f41afe8814fd61b6d460ce161ef90ce6e 100644 (file)
@@ -144,7 +144,7 @@ logicalrep_relmap_free_entry(LogicalRepRelMapEntry *entry)
    bms_free(remoterel->attkeys);
 
    if (entry->attrmap)
-       pfree(entry->attrmap);
+       free_attrmap(entry->attrmap);
 }
 
 /*
@@ -373,6 +373,13 @@ logicalrep_rel_open(LogicalRepRelId remoteid, LOCKMODE lockmode)
        int         i;
        Bitmapset  *missingatts;
 
+       /* Release the no-longer-useful attrmap, if any. */
+       if (entry->attrmap)
+       {
+           free_attrmap(entry->attrmap);
+           entry->attrmap = NULL;
+       }
+
        /* Try to find and lock the relation by name. */
        relid = RangeVarGetRelid(makeRangeVar(remoterel->nspname,
                                              remoterel->relname, -1),
@@ -620,6 +627,13 @@ logicalrep_partition_open(LogicalRepRelMapEntry *root,
        part_entry->partoid = partOid;
    }
 
+   /* Release the no-longer-useful attrmap, if any. */
+   if (entry->attrmap)
+   {
+       free_attrmap(entry->attrmap);
+       entry->attrmap = NULL;
+   }
+
    if (!entry->remoterel.remoteid)
    {
        int         i;