kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2026-07-17 00:47:44 -07:00
Code Issues Packages Projects Releases Wiki Activity

Merge "pbx: Fix leak of timezone for time based includes." into 11

Browse Source
This commit is contained in:
zuul
2016-07-14 12:05:22 -05:00
committed by Gerrit Code Review
parent a8775bc6df 877c23222d
commit 39f1c269cb
1 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
main/pbx.c
+12 -7
View File
@@ -7119,6 +7119,13 @@ static struct ast_context *find_context_locked(const char *context)
return c;
}
/*! \brief Free an ast_include and associated data. */
static void include_free(struct ast_include *include)
{
ast_destroy_timing(&(include->timing));
ast_free(include);
}
/*!
* \brief Remove included contexts.
* This function locks contexts list by &conlist, search for the right context
@@ -7166,8 +7173,7 @@ int ast_context_remove_include2(struct ast_context *con, const char *include, co
else
con->includes = i->next;
/* free include and return */
ast_destroy_timing(&(i->timing));
ast_free(i);
include_free(i);
ret = 0;
break;
}
@@ -9595,8 +9601,7 @@ int ast_context_add_include2(struct ast_context *con, const char *value,
/* ... go to last include and check if context is already included too... */
for (i = con->includes; i; i = i->next) {
if (!strcasecmp(i->name, new_include->name)) {
ast_destroy_timing(&(new_include->timing));
ast_free(new_include);
include_free(new_include);
ast_unlock_context(con);
errno = EEXIST;
return -1;
@@ -10926,7 +10931,7 @@ static void __ast_internal_context_destroy( struct ast_context *con)
for (tmpi = tmp->includes; tmpi; ) { /* Free includes */
struct ast_include *tmpil = tmpi;
tmpi = tmpi->next;
ast_free(tmpil);
include_free(tmpil);
}
for (ipi = tmp->ignorepats; ipi; ) { /* Free ignorepats */
struct ast_ignorepat *ipl = ipi;
@@ -11020,12 +11025,12 @@ void __ast_context_destroy(struct ast_context *list, struct ast_hashtab *context
if (pi) {
pi->next = i->next;
/* free include */
ast_free(i);
include_free(i);
continue; /* don't change pi */
} else {
tmp->includes = i->next;
/* free include */
ast_free(i);
include_free(i);
continue; /* don't change pi */
}
}