[PATCH] fetch.c: Plug memory leak in process_tree()

sigprof · Junio C Hamano · commit d35bbe0b2e37 · 2005-09-23T14:30:45.000-07:00
When freeing a tree entry, must free its name too.

Signed-off-by: Sergey Vlasov &lt;vsu@altlinux.ru&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
diff --git a/fetch.c b/fetch.c
@@ -48,6 +48,7 @@ static int process_tree(struct tree *tree)
 		struct tree_entry_list *next = entry->next;
 		if (process(entry->item.any))
 			return -1;
+		free(entry->name);
 		free(entry);
 		entry = next;
 	}

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,7 @@ static int process_tree(struct tree *tree)`
`48`	`48`	`struct tree_entry_list *next = entry->next;`
`49`	`49`	`if (process(entry->item.any))`
`50`	`50`	`return -1;`
	`51`	`+ free(entry->name);`
`51`	`52`	`free(entry);`
`52`	`53`	`entry = next;`
`53`	`54`	`}`