In response to 10578, I just ran 1.4 thru valgrind; some of the config leakage I've already fixed, but it doesn't hurt to double check. I found and fixed leaks in res_jabber, cdr_tds, pbx_ael. Nothing major, tho.

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@89088 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Steve Murphy
2007-11-07 21:40:28 +00:00
parent 63f6e6b023
commit 164d8a5e61
3 changed files with 10 additions and 4 deletions

View File

@@ -441,9 +441,11 @@ static int tds_load_module(void)
}
var = ast_variable_browse(cfg, "global");
if (!var) /* nothing configured */
if (!var) /* nothing configured */ {
ast_config_destroy(cfg);
return 0;
}
ptr = ast_variable_retrieve(cfg, "global", "hostname");
if (ptr)
hostname = strdup(ptr);

View File

@@ -2993,7 +2993,7 @@ static void gen_prios(struct ael_extension *exten, char *label, pval *statement,
strncat(buf2,strp2+1, sizeof(buf2)-strlen(strp2+1)-2);
strcat(buf2,"]");
for_init->appargs = strdup(buf2);
for_init->app = strdup("Set");
/* for_init->app = strdup("Set"); just set! */
} else {
strp2 = p->u1.for_init;
while (*strp2 && isspace(*strp2))
@@ -3015,7 +3015,8 @@ static void gen_prios(struct ael_extension *exten, char *label, pval *statement,
*strp3 = 0; /* remove the closing paren */
for_init->appargs = strdup(buf2);
if (for_init->app)
free(for_init->app);
for_init->app = strdup("Macro");
} else { /* must be a regular app call */
char *strp3;
@@ -3023,6 +3024,8 @@ static void gen_prios(struct ael_extension *exten, char *label, pval *statement,
strp3 = strchr(buf2,'(');
if (strp3) {
*strp3 = 0;
if (for_init->app)
free(for_init->app);
for_init->app = strdup(buf2);
for_init->appargs = strdup(strp3+1);
strp3 = strrchr(for_init->appargs, ')');

View File

@@ -2354,6 +2354,7 @@ static int aji_load_config(void)
}
cat = ast_category_browse(cfg, cat);
}
ast_config_destroy(cfg); /* or leak memory */
return 1;
}