test_bitmap_walk: free bitmap with bitmap_free

peff · gitster · commit d201a1ecdb12 · 2015-05-22T09:03:04.000-07:00
Commit f86a374 (pack-bitmap.c: fix a memleak, 2015-03-30) noticed that we leak the "result" bitmap. But we should use "bitmap_free" rather than straight "free", as the former remembers to free the bitmap array pointed to by the struct. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
diff --git a/pack-bitmap.c b/pack-bitmap.c
@@ -987,7 +987,7 @@ void test_bitmap_walk(struct rev_info *revs)
 	else
 		fprintf(stderr, "Mismatch!\n");
 
-	free(result);
+	bitmap_free(result);
 }
 
 static int rebuild_bitmap(uint32_t *reposition,

Original file line number	Diff line number	Diff line change
`@@ -987,7 +987,7 @@ void test_bitmap_walk(struct rev_info *revs)`
`987`	`987`	`else`
`988`	`988`	`fprintf(stderr, "Mismatch!\n");`
`989`	`989`
`990`		`- free(result);`
	`990`	`+ bitmap_free(result);`
`991`	`991`	`}`
`992`	`992`
`993`	`993`	`static int rebuild_bitmap(uint32_t *reposition,`