From 9094d30b932ca4b47dba81e76011efe05455a44a Mon Sep 17 00:00:00 2001 From: Sascha Cunz Date: Fri, 23 Nov 2012 11:41:56 +0100 Subject: Reset all static variables to NULL in clar's __cleanup Without this change, any failed assertion in the second (or a later) test inside a test suite has a chance of double deleting memory, resulting in a heap corruption. See #1096 for details. This leaves alone the test cases where we "just" use cl_git_sandbox_init() and cl_git_sandbox_cleanup(). These methods already take good care to not double delete a repository. Fixes #1096 --- tests-clar/diff/blob.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'tests-clar/diff/blob.c') diff --git a/tests-clar/diff/blob.c b/tests-clar/diff/blob.c index 0a37e829d..a9ebcc207 100644 --- a/tests-clar/diff/blob.c +++ b/tests-clar/diff/blob.c @@ -30,7 +30,10 @@ void test_diff_blob__initialize(void) void test_diff_blob__cleanup(void) { git_blob_free(d); + d = NULL; + git_blob_free(alien); + alien = NULL; cl_git_sandbox_cleanup(); } -- cgit v1.2.1