Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit fbcc9fe

Browse files
committed
Don't leak rd_statlist when a relcache entry is dropped.
Although these lists are usually NIL, and even when not empty are unlikely to be large, constant relcache update traffic could eventually result in visible bloat of CacheMemoryContext. Found via valgrind testing. Back-patch to v10 where this field was added. Discussion: https://postgr.es/m/3816764.1616104288@sss.pgh.pa.us
1 parent a5eb70c commit fbcc9fe

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

src/backend/utils/cache/relcache.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2348,6 +2348,7 @@ RelationDestroyRelation(Relation relation, bool remember_tupdesc)
23482348
FreeTriggerDesc(relation->trigdesc);
23492349
list_free_deep(relation->rd_fkeylist);
23502350
list_free(relation->rd_indexlist);
2351+
list_free(relation->rd_statlist);
23512352
bms_free(relation->rd_indexattr);
23522353
bms_free(relation->rd_keyattr);
23532354
bms_free(relation->rd_pkattr);

0 commit comments

Comments
 (0)