fix: memory and resource leaks

ASTERISK-26119 #close

Change-Id: Iecbf7d0f360a021147344c4e83ab242fd1e7512c
changes/50/3050/1
Alexei Gradinari 9 years ago
parent 0c69337405
commit 7133572cf8

@ -3668,13 +3668,20 @@ op_tildetilde (struct val *a, struct val *b)
/* strip double quotes from both -- */
strip_quotes(a);
strip_quotes(b);
vs = malloc(strlen(a->u.s)+strlen(b->u.s)+1);
if (vs == NULL) {
ast_log(LOG_WARNING, "malloc() failed\n");
return NULL;
}
strcpy(vs,a->u.s);
strcat(vs,b->u.s);
v = make_str(vs);
free(vs);
/* free arguments */
free_value(a);
free_value(b);

@ -1661,13 +1661,20 @@ op_tildetilde (struct val *a, struct val *b)
/* strip double quotes from both -- */
strip_quotes(a);
strip_quotes(b);
vs = malloc(strlen(a->u.s)+strlen(b->u.s)+1);
if (vs == NULL) {
ast_log(LOG_WARNING, "malloc() failed\n");
return NULL;
}
strcpy(vs,a->u.s);
strcat(vs,b->u.s);
v = make_str(vs);
free(vs);
/* free arguments */
free_value(a);
free_value(b);

@ -3356,9 +3356,9 @@ static int gen_prios(struct ael_extension *exten, char *label, pval *statement,
#ifdef OLD_RAND_ACTION
struct ael_priority *rand_test, *rand_end, *rand_skip;
#endif
char *buf1;
char *buf2;
char *new_label;
RAII_VAR(char *, buf1, NULL, free);
RAII_VAR(char *, buf2, NULL, free);
RAII_VAR(char *, new_label, NULL, free);
char *strp, *strp2;
int default_exists;
int local_control_statement_count;
@ -4216,9 +4216,6 @@ static int gen_prios(struct ael_extension *exten, char *label, pval *statement,
break;
}
}
free(buf1);
free(buf2);
free(new_label);
return 0;
}
@ -5081,7 +5078,10 @@ int pvalCheckType( pval *p, char *funcname, pvaltype type )
pval *pvalCreateNode( pvaltype type )
{
pval *p = calloc(1,sizeof(pval)); /* why, oh why, don't I use ast_calloc? Way, way, way too messy if I do! */
p->type = type; /* remember, this can be used externally or internally to asterisk */
/* remember, this can be used externally or internally to asterisk */
if (p) {
p->type = type;
}
return p;
}
@ -5442,14 +5442,30 @@ void pvalIncludesAddInclude( pval *p, const char *include )
void pvalIncludesAddIncludeWithTimeConstraints( pval *p, const char *include, char *hour_range, char *dom_range, char *dow_range, char *month_range )
{
pval *hr = pvalCreateNode(PV_WORD);
pval *dom = pvalCreateNode(PV_WORD);
pval *dow = pvalCreateNode(PV_WORD);
pval *mon = pvalCreateNode(PV_WORD);
pval *s = pvalCreateNode(PV_WORD);
if (!pvalCheckType(p, "pvalIncludeAddIncludeWithTimeConstraints", PV_INCLUDES))
pval *hr;
pval *dom;
pval *dow;
pval *mon;
pval *s;
if (!pvalCheckType(p, "pvalIncludeAddIncludeWithTimeConstraints", PV_INCLUDES)) {
return;
}
hr = pvalCreateNode(PV_WORD);
dom = pvalCreateNode(PV_WORD);
dow = pvalCreateNode(PV_WORD);
mon = pvalCreateNode(PV_WORD);
s = pvalCreateNode(PV_WORD);
if (!hr || !dom || !dow || !mon || !s) {
destroy_pval(hr);
destroy_pval(dom);
destroy_pval(dow);
destroy_pval(mon);
destroy_pval(s);
return;
}
s->u1.str = (char *)include;
p->u1.list = linku1(p->u1.list, s);
@ -5696,12 +5712,28 @@ char* pvalIfGetCondition( pval *p )
void pvalIfTimeSetCondition( pval *p, char *hour_range, char *dow_range, char *dom_range, char *mon_range ) /* time range format: 24-hour format begin-end|dow range|dom range|month range */
{
pval *hr = pvalCreateNode(PV_WORD);
pval *dow = pvalCreateNode(PV_WORD);
pval *dom = pvalCreateNode(PV_WORD);
pval *mon = pvalCreateNode(PV_WORD);
if (!pvalCheckType(p, "pvalIfTimeSetCondition", PV_IFTIME))
pval *hr;
pval *dow;
pval *dom;
pval *mon;
if (!pvalCheckType(p, "pvalIfTimeSetCondition", PV_IFTIME)) {
return;
}
hr = pvalCreateNode(PV_WORD);
dow = pvalCreateNode(PV_WORD);
dom = pvalCreateNode(PV_WORD);
mon = pvalCreateNode(PV_WORD);
if (!hr || !dom || !dow || !mon) {
destroy_pval(hr);
destroy_pval(dom);
destroy_pval(dow);
destroy_pval(mon);
return;
}
pvalWordSetString(hr, hour_range);
pvalWordSetString(dow, dow_range);
pvalWordSetString(dom, dom_range);

@ -327,10 +327,13 @@ static int load_file(const char *filename, char **ret)
fseek(f, 0, SEEK_END);
len = ftell(f);
fseek(f, 0, SEEK_SET);
if (!(*ret = ast_malloc(len + 1)))
if (!(*ret = ast_malloc(len + 1))) {
fclose(f);
return -2;
}
if (len != fread(*ret, sizeof(char), len, f)) {
fclose(f);
free(*ret);
*ret = NULL;
return -3;

Loading…
Cancel
Save