Fixed nasty bug in error handling. It should have been setting the return

bytecode value to NULL on error, but wasn't (random return value is bad!).

svn path=/trunk/yasm/; revision=221

diff --git a/modules/parsers/nasm/gen_instr.pl b/modules/parsers/nasm/gen_instr.pl
index 3561a3e8ac405d4b65fdda223e9bf6bd65d9bb81..5b933de43987262ae58eef974840db85110ae86a 100755 (executable)
--- a/modules/parsers/nasm/gen_instr.pl
+++ b/modules/parsers/nasm/gen_instr.pl
@@ -810,6 +810,7 @@ sub output_yacc ($@)
                # ASSUMES: at least one previous action exists
                print GRAMMAR "    | \Ugrp_$group\E error {\n";
                print GRAMMAR "        Error (_(\"expression syntax error\"));\n";
+               print GRAMMAR "        \$\$ = (bytecode *)NULL\n";
                print GRAMMAR "    }\n";
 
                # terminate the rule

diff --git a/src/parsers/nasm/gen_instr.pl b/src/parsers/nasm/gen_instr.pl
index 3561a3e8ac405d4b65fdda223e9bf6bd65d9bb81..5b933de43987262ae58eef974840db85110ae86a 100755 (executable)
--- a/src/parsers/nasm/gen_instr.pl
+++ b/src/parsers/nasm/gen_instr.pl
@@ -810,6 +810,7 @@ sub output_yacc ($@)
                # ASSUMES: at least one previous action exists
                print GRAMMAR "    | \Ugrp_$group\E error {\n";
                print GRAMMAR "        Error (_(\"expression syntax error\"));\n";
+               print GRAMMAR "        \$\$ = (bytecode *)NULL\n";
                print GRAMMAR "    }\n";
 
                # terminate the rule