diff options
author | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-01-03 16:34:12 +0000 |
---|---|---|
committer | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-01-03 16:34:12 +0000 |
commit | ccf5352c19d37cb8f227e226d0a5ca15b2910430 (patch) | |
tree | b536cedf37c823f468156393376cb7999e9a940d /ast_expr2f.c | |
parent | 65e3e8093cc5767d990a28fdae89b63a1a187105 (diff) |
don't leak memory for (most) expression evaluations
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.2@7736 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'ast_expr2f.c')
-rw-r--r-- | ast_expr2f.c | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/ast_expr2f.c b/ast_expr2f.c index defaa100c..71f62ee0b 100644 --- a/ast_expr2f.c +++ b/ast_expr2f.c @@ -2562,42 +2562,42 @@ int ast_yyerror(const char *, YYLTYPE *, struct parse_io *); /* likewise */ int ast_expr(char *expr, char *buf, int length) { - struct parse_io *io; + struct parse_io io; + int return_value = 0; - io = calloc(sizeof(struct parse_io),1); - io->string = expr; /* to pass to the error routine */ + memset(&io, 0, sizeof(io)); + io.string = expr; /* to pass to the error routine */ - ast_yylex_init(&io->scanner); + ast_yylex_init(&io.scanner); - ast_yy_scan_string(expr, io->scanner); + ast_yy_scan_string(expr, io.scanner); - ast_yyparse ((void *) io); + ast_yyparse ((void *) &io); - ast_yylex_destroy(io->scanner); + ast_yylex_destroy(io.scanner); - if (io->val == NULL) { + if (!io.val) { if (length > 1) { strcpy(buf, "0"); - return 1; + return_value = 1; } } else { - if (io->val->type == AST_EXPR_integer) { + if (io.val->type == AST_EXPR_integer) { int res_length; - res_length = snprintf(buf, length, "%ld", (long int) io->val->u.i); - return res_length <= length ? res_length : length; + res_length = snprintf(buf, length, "%ld", (long int) io.val->u.i); + return_value = (res_length <= length) ? res_length : length; } else { #ifdef STANDALONE - strncpy(buf, io->val->u.s, length - 1); + strncpy(buf, io.val->u.s, length - 1); #else /* !STANDALONE */ - ast_copy_string(buf, io->val->u.s, length); + ast_copy_string(buf, io.val->u.s, length); #endif /* STANDALONE */ - return strlen(buf); + return_value = strlen(buf); } - free(io->val); + free(io.val); } - free(io); - return 0; + return return_value; } int ast_yyerror (const char *s, yyltype *loc, struct parse_io *parseio ) |