Merge pull request #121 from Brainrotlang/ci/memoryl_leaks

ci(memory_leaks): Fix CI valgrind check and resolve function memory leaks

Author: SIGMazer

ast.c

@@ -3435,6 +3435,7 @@ void free_ast(ASTNode *node)
     case NODE_FUNCTION_DEF:
         SAFE_FREE(node->data.function_def.name);
         // Free parameters
+        free_parameters(node->data.function_def.parameters);
         if (node->data.function_def.body)
         {
             free_ast(node->data.function_def.body);
@@ -3786,6 +3787,7 @@ void enter_function_scope(Function *func, ArgumentList *args)
         Variable *var = variable_new(curr_param->name);
         var->var_type = curr_param->type;
         add_variable_to_scope(curr_param->name, var);
+        SAFE_FREE(var);
 
         switch (curr_param->type)
         {
@@ -3810,5 +3812,6 @@ void enter_function_scope(Function *func, ArgumentList *args)
         }
         curr_param = curr_param->next;
     }
+    reverse_parameter_list(&func->parameters);
 }
 

run_valgrind_tests.sh

@@ -18,5 +18,9 @@ for f in test_cases/*.brainrot; do
     else
         valgrind --leak-check=full --error-exitcode=1 ./brainrot "$f"
     fi
+    if [[ $? -ne 0 ]]; then
+        echo "Valgrind failed on $f"
+        exit 1
+    fi
     echo
-done
+done