Merge pull request verilog-to-routing#685 from j-b-1-7/odin_leak_get_name_of_pins_rnode_1

jeanlego · web-flow · commit 0735cae1cba6 · 2019-07-04T00:42:37.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
@@ -720,7 +720,12 @@ char_list_t *get_name_of_pins(ast_node_t *var_node, char *instance_name_prefix)
 						i+rnode[2]->types.vnumber->get_value());
 					index++;
 				}
+				if(rnode[1] != sym_node->children[1])
+				{
+					free_whole_tree(rnode[1]);
+				}
 			}
+
 			else if (sym_node->children[3] != NULL)
 			{
 				oassert(FALSE);
@@ -1628,4 +1633,5 @@ long resolve_concat_sizes(ast_node_t *node_top, char *instance_name_prefix, STRI
 	}
 
 	return concatenation_size;
-}
+}
+

Original file line number	Diff line number	Diff line change
`@@ -720,7 +720,12 @@ char_list_t get_name_of_pins(ast_node_t var_node, char *instance_name_prefix)`
`720`	`720`	`i+rnode[2]->types.vnumber->get_value());`
`721`	`721`	`index++;`
`722`	`722`	`}`
	`723`	`+ if(rnode[1] != sym_node->children[1])`
	`724`	`+ {`
	`725`	`+ free_whole_tree(rnode[1]);`
	`726`	`+ }`
`723`	`727`	`}`
	`728`	`+`
`724`	`729`	`else if (sym_node->children[3] != NULL)`
`725`	`730`	`{`
`726`	`731`	`oassert(FALSE);`
`@@ -1628,4 +1633,5 @@ long resolve_concat_sizes(ast_node_t node_top, char instance_name_prefix, STRI`
`1628`	`1633`	`}`
`1629`	`1634`
`1630`	`1635`	`return concatenation_size;`
`1631`		`-}`
	`1636`	`+}`
	`1637`	`+`