Merge pull request #683 from j-b-1-7/odin_leak_get_name_of_pins_sym_node

jeanlego · web-flow · commit 9bbbc66811d4 · 2019-07-04T00:23:44.000-03:00
Fixed memory leak in get_name_of_pins function in ast_util.cpp
diff --git a/ODIN_II/SRC/ast_util.cpp b/ODIN_II/SRC/ast_util.cpp
@@ -731,6 +731,8 @@ char_list_t *get_name_of_pins(ast_node_t *var_node, char *instance_name_prefix)
 			oassert(sym_node->type == NUMBERS);
 			width = sym_node->types.vnumber->size();
 			return_string = get_name_of_pins_number(sym_node, 0, width);
+
+			free_whole_tree(sym_node);
 		}
 	}
 	else if (var_node->type == NUMBERS)

Original file line number	Diff line number	Diff line change
`@@ -731,6 +731,8 @@ char_list_t get_name_of_pins(ast_node_t var_node, char *instance_name_prefix)`
`731`	`731`	`oassert(sym_node->type == NUMBERS);`
`732`	`732`	`width = sym_node->types.vnumber->size();`
`733`	`733`	`return_string = get_name_of_pins_number(sym_node, 0, width);`
	`734`	`+`
	`735`	`+ free_whole_tree(sym_node);`
`734`	`736`	`}`
`735`	`737`	`}`
`736`	`738`	`else if (var_node->type == NUMBERS)`