Fix memory leak in logger.
authorKevin Harwell <kharwell@digium.com>
Fri, 20 Sep 2013 14:26:44 +0000 (14:26 +0000)
committerKevin Harwell <kharwell@digium.com>
Fri, 20 Sep 2013 14:26:44 +0000 (14:26 +0000)
Fixed a memory leak discovered in the logger where a temporary string buffer
was not being freed.

(closes issue ASTERISK-22540)
Reported by: John Hardin
........

Merged revisions 399513 from http://svn.asterisk.org/svn/asterisk/branches/11
........

Merged revisions 399514 from http://svn.asterisk.org/svn/asterisk/branches/12

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@399515 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/logger.c

index 1124568..a6b25cd 100644 (file)
@@ -1632,6 +1632,7 @@ void __ast_verbose_ap(const char *file, int line, const char *func, int level, s
        res = ast_str_set_va(&buf, 0, fmt, ap);
        /* If the build failed then we can drop this allocated message */
        if (res == AST_DYNSTR_BUILD_FAILED) {
+               ast_free(buf);
                return;
        }
 
@@ -1659,6 +1660,7 @@ void __ast_verbose_ap(const char *file, int line, const char *func, int level, s
        } while (p && *p);
 
        ast_log_callid(__LOG_VERBOSE, file, line, func, callid, "%s", ast_str_buffer(prefixed));
+       ast_free(buf);
 }
 
 void __ast_verbose(const char *file, int line, const char *func, int level, const char *fmt, ...)