grep: Fix two memory leaks

toofishes · gitster · commit bfac23d9534e · 2010-01-30T09:11:00.000-08:00
We duplicate the grep_opt structure when using grep threads, but didn't
later free either the patterns attached to this new structure or the
structure itself.

Signed-off-by: Dan McGee &lt;dpmcgee@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/builtin-grep.c b/builtin-grep.c
@@ -205,6 +205,8 @@ static void *run(void *arg)
 
 		work_done(w);
 	}
+	free_grep_patterns(arg);
+	free(arg);
 
 	return (void*) (intptr_t) hit;
 }

Original file line number	Diff line number	Diff line change
`@@ -205,6 +205,8 @@ static void run(void arg)`
`205`	`205`
`206`	`206`	`work_done(w);`
`207`	`207`	`}`
	`208`	`+ free_grep_patterns(arg);`
	`209`	`+ free(arg);`
`208`	`210`
`209`	`211`	`return (void*) (intptr_t) hit;`
`210`	`212`	`}`