Merged revisions 84239 via svnmerge from
authorSteve Murphy <murf@digium.com>
Mon, 1 Oct 2007 23:03:50 +0000 (23:03 +0000)
committerSteve Murphy <murf@digium.com>
Mon, 1 Oct 2007 23:03:50 +0000 (23:03 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r84239 | murf | 2007-10-01 14:27:52 -0600 (Mon, 01 Oct 2007) | 1 line

closes issue #10777 -- by returning a null for the parse tree when there's really nothing there, and making sure we don't try to do checking on a null tree.
........

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

pbx/pbx_ael.c
res/ael/ael.tab.c
res/ael/ael.y
res/ael/pval.c

index 8f4ae14..68902a0 100644 (file)
@@ -905,7 +905,7 @@ static int aeldebug = 0;
 
 static int pbx_load_module(void)
 {
-       int errs, sem_err, sem_warn, sem_note;
+       int errs=0, sem_err=0, sem_warn=0, sem_note=0;
        char *rfilename;
        struct ast_context *local_contexts=NULL, *con;
        struct pval *parse_tree;
index c6d2ce4..1864c18 100644 (file)
@@ -2137,8 +2137,8 @@ yyreduce:
                if (!(yyvsp[(5) - (6)].pval)) {
                         ast_log(LOG_WARNING, "==== File: %s, Line %d, Cols: %d-%d: Warning! The empty context %s will be IGNORED!\n", 
                                my_file, (yylsp[(4) - (6)]).first_line, (yylsp[(4) - (6)]).first_column, (yylsp[(4) - (6)]).last_column, (yyvsp[(3) - (6)].str) );
+                       (yyval.pval) = 0;
                        free((yyvsp[(3) - (6)].str));
-
                } else {
                        (yyval.pval) = npval2(PV_CONTEXT, &(yylsp[(1) - (6)]), &(yylsp[(6) - (6)]));
                        (yyval.pval)->u1.str = (yyvsp[(3) - (6)].str);
index ed48d48..42d86c0 100644 (file)
@@ -205,8 +205,8 @@ context : opt_abstract KW_CONTEXT context_name LC elements RC {
                if (!$5) {
                         ast_log(LOG_WARNING, "==== File: %s, Line %d, Cols: %d-%d: Warning! The empty context %s will be IGNORED!\n", 
                                my_file, @4.first_line, @4.first_column, @4.last_column, $3 );
+                       $$ = 0;
                        free($3);
-
                } else {
                        $$ = npval2(PV_CONTEXT, &@1, &@6);
                        $$->u1.str = $3;
index 11c7253..960c3ec 100644 (file)
@@ -2871,6 +2871,8 @@ void ael2_semantic_check(pval *item, int *arg_errs, int *arg_warns, int *arg_not
 #endif
        struct argapp *apps=0;
 
+       if (!item)
+               return; /* don't check an empty tree */
 #ifdef AAL_ARGCHECK
        rfilename = alloca(10 + strlen(ast_config_AST_VAR_DIR));
        sprintf(rfilename, "%s/applist", ast_config_AST_VAR_DIR);