attr: fix leak in free_attr_elem

peff · gitster · commit 37475f97d1fb · 2012-01-11T19:07:23.000-08:00
This function frees the individual "struct match_attr"s we
have allocated, but forgot to free the array holding their
pointers, leading to a minor memory leak (but it can add up
after checking attributes for paths in many directories).

Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/attr.c b/attr.c
@@ -286,6 +286,7 @@ static void free_attr_elem(struct attr_stack *e)
 		}
 		free(a);
 	}
+	free(e->attrs);
 	free(e);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -286,6 +286,7 @@ static void free_attr_elem(struct attr_stack *e)`
`286`	`286`	`}`
`287`	`287`	`free(a);`
`288`	`288`	`}`
	`289`	`+ free(e->attrs);`
`289`	`290`	`free(e);`
`290`	`291`	`}`
`291`	`292`