Fix memory leaks in event trigger code.

Spotted by Jeff Davis.
This commit is contained in:
Robert Haas 2012-08-07 16:59:42 -04:00
parent ac78c4178b
commit eea65943c6
1 changed files with 4 additions and 3 deletions

View File

@ -84,7 +84,7 @@ BuildEventTriggerCache(void)
* the cache failed, then there might be some junk lying around * the cache failed, then there might be some junk lying around
* that we want to reclaim. * that we want to reclaim.
*/ */
MemoryContextReset(EventTriggerCacheContext); MemoryContextResetAndDeleteChildren(EventTriggerCacheContext);
} }
else else
{ {
@ -119,8 +119,9 @@ BuildEventTriggerCache(void)
ctl.keysize = sizeof(EventTriggerEvent); ctl.keysize = sizeof(EventTriggerEvent);
ctl.entrysize = sizeof(EventTriggerCacheEntry); ctl.entrysize = sizeof(EventTriggerCacheEntry);
ctl.hash = tag_hash; ctl.hash = tag_hash;
ctl.hcxt = EventTriggerCacheContext;
cache = hash_create("Event Trigger Cache", 32, &ctl, cache = hash_create("Event Trigger Cache", 32, &ctl,
HASH_ELEM | HASH_FUNCTION); HASH_ELEM | HASH_FUNCTION | HASH_CONTEXT);
/* /*
* Prepare to scan pg_event_trigger in name order. We use an MVCC * Prepare to scan pg_event_trigger in name order. We use an MVCC
@ -237,6 +238,6 @@ DecodeTextArrayToCString(Datum array, char ***cstringp)
static void static void
InvalidateEventCacheCallback(Datum arg, int cacheid, uint32 hashvalue) InvalidateEventCacheCallback(Datum arg, int cacheid, uint32 hashvalue)
{ {
MemoryContextReset(EventTriggerCacheContext); MemoryContextResetAndDeleteChildren(EventTriggerCacheContext);
EventTriggerCache = NULL; EventTriggerCache = NULL;
} }