From: Wolfgang (Blub) Bumiller <blub@speed.at>
Date: Tue, 21 Aug 2012 09:18:40 +0000 (+0200)
Subject: some more error situations where typevar wouldn't get freed
X-Git-Tag: 0.1-rc1~124
X-Git-Url: https://git.rm.cloudns.org/?a=commitdiff_plain;h=d7f43dfa4dda33e7c2d481259fa2a5e14e0390db;p=xonotic%2Fgmqcc.git

some more error situations where typevar wouldn't get freed
---

diff --git a/parser.c b/parser.c
index 4874f49..a3241c8 100644
--- a/parser.c
+++ b/parser.c
@@ -2006,6 +2006,7 @@ static bool parser_variable(parser_t *parser, ast_block *localblock)
             fval = ast_value_new(ctx, var->name, TYPE_FUNCTION);
             if (!fval) {
                 ast_value_delete(var);
+                ast_value_delete(typevar);
                 if (fval) ast_value_delete(fval);
                 return false;
             }
@@ -2023,6 +2024,7 @@ static bool parser_variable(parser_t *parser, ast_block *localblock)
                                proto->name,
                                ast_ctx(proto).file, ast_ctx(proto).line);
                     ast_value_delete(fval);
+                    ast_value_delete(typevar);
                     return false;
                 }
                 /* copy over the parameter names */
@@ -2048,6 +2050,7 @@ static bool parser_variable(parser_t *parser, ast_block *localblock)
                 varentry_t vx, vy, vz;
                 if (!create_vector_members(parser, var, &vx, &vy, &vz)) {
                     ast_delete(var);
+                    ast_value_delete(typevar);
                     return false;
                 }
 
@@ -2071,6 +2074,7 @@ static bool parser_variable(parser_t *parser, ast_block *localblock)
                         parser_pop_local(parser);
                         parser_pop_local(parser);
                         ast_value_delete(var);
+                        ast_value_delete(typevar);
                         return false;
                     }
                 }
@@ -2081,12 +2085,14 @@ static bool parser_variable(parser_t *parser, ast_block *localblock)
                      ( localblock && !parser_t_locals_add(parser, varent)) )
                 {
                     ast_value_delete(var);
+                    ast_value_delete(typevar);
                     return false;
                 }
                 if (localblock && !ast_block_locals_add(localblock, var))
                 {
                     parser_pop_local(parser);
                     ast_value_delete(var);
+                    ast_value_delete(typevar);
                     return false;
                 }
             }