Merge pull request #684 from j-b-1-7/odin_leak_assignment_alias_right

jeanlego · web-flow · commit a979e268af9d · 2019-07-04T00:24:43.000-03:00
Fixed memory leak in assignment_alias function in netlist_creat_…
diff --git a/ODIN_II/SRC/netlist_create_from_ast.cpp b/ODIN_II/SRC/netlist_create_from_ast.cpp
@@ -3326,11 +3326,18 @@ signal_list_t *assignment_alias(ast_node_t* assignment, char *instance_name_pref
 	}
 	else
 	{
+
+		ast_node_t *temp = right;
 		long assignment_size = get_size_of_variable(left, instance_name_prefix, local_symbol_table_sc, function_local_symbol_table_sc);
 		right = resolve_node(NULL, instance_name_prefix, right, NULL, assignment_size);
 
 		in_1 = netlist_expand_ast_of_module(right, instance_name_prefix);
 		oassert(in_1 != NULL);
+
+		if(right != temp)
+		{
+			free_whole_tree(right);
+		}
 	}
 
 	char_list_t *out_list;

Original file line number	Diff line number	Diff line change
`@@ -3326,11 +3326,18 @@ signal_list_t assignment_alias(ast_node_t assignment, char *instance_name_pref`
`3326`	`3326`	`}`
`3327`	`3327`	`else`
`3328`	`3328`	`{`
	`3329`	`+`
	`3330`	`+ ast_node_t *temp = right;`
`3329`	`3331`	`long assignment_size = get_size_of_variable(left, instance_name_prefix, local_symbol_table_sc, function_local_symbol_table_sc);`
`3330`	`3332`	`right = resolve_node(NULL, instance_name_prefix, right, NULL, assignment_size);`
`3331`	`3333`
`3332`	`3334`	`in_1 = netlist_expand_ast_of_module(right, instance_name_prefix);`
`3333`	`3335`	`oassert(in_1 != NULL);`
	`3336`	`+`
	`3337`	`+ if(right != temp)`
	`3338`	`+ {`
	`3339`	`+ free_whole_tree(right);`
	`3340`	`+ }`
`3334`	`3341`	`}`
`3335`	`3342`
`3336`	`3343`	`char_list_t *out_list;`