diff options
author | Richard Mudgett <rmudgett@digium.com> | 2016-06-01 13:57:53 -0500 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2016-06-01 14:09:36 -0500 |
commit | 40d19f2e5531fd3455e21a51e39a7244580609bb (patch) | |
tree | ab5979f47992da392141afa2e527a95cc22bf386 /cdr/cdr_syslog.c | |
parent | 608e0267e8512e16e186cbdc874cd7902e287ff7 (diff) |
logging,cdr,cel: Fix stringfield memory leak.
The stringfields refactor to allow adding stringfields to the end of a
structure (f6f4cf459f43f072604927209b39646f84aaa2e2) exposed some
incomplete cleanup code by some stringfield users.
The most noticeable leaker is the logging system where there is a leak for
every log message generated.
ASTERISK-26078 #close
Reported by: Etienne Lessard
Patches:
jira_asterisk_26078_v13.patch (license #5621) patch uploaded
by Richard Mudgett
Change-Id: If6a08b31336b492c3de6f9dfd07c447f8d5a8782
Diffstat (limited to 'cdr/cdr_syslog.c')
-rw-r--r-- | cdr/cdr_syslog.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/cdr/cdr_syslog.c b/cdr/cdr_syslog.c index 85860ecec..39f115920 100644 --- a/cdr/cdr_syslog.c +++ b/cdr/cdr_syslog.c @@ -76,8 +76,10 @@ static AST_RWLIST_HEAD_STATIC(sinks, cdr_syslog_config); static void free_config(void) { struct cdr_syslog_config *sink; + while ((sink = AST_RWLIST_REMOVE_HEAD(&sinks, list))) { ast_mutex_destroy(&sink->lock); + ast_string_field_free_memory(sink); ast_free(sink); } } |