]> granicus.if.org Git - p11-kit/commitdiff
test: Fix memleak in test-token cleanup
authorDaiki Ueno <dueno@redhat.com>
Tue, 20 Sep 2016 14:48:16 +0000 (16:48 +0200)
committerDaiki Ueno <ueno@gnu.org>
Mon, 31 Oct 2016 14:24:17 +0000 (15:24 +0100)
GCC's asan spotted this:
  Direct leak of 338 byte(s) in 13 object(s) allocated from:
      #0 0x7f54f03fee20 in malloc (/lib64/libasan.so.3+0xc6e20)
      #1 0x445e8c in p11_path_build ../common/path.c:222
      #2 0x4385bd in expand_tempdir ../common/test.c:334
      #3 0x43869c in p11_test_directory ../common/test.c:361
      #4 0x4033e3 in setup_temp ../trust/test-token.c:79

trust/test-token.c

index d4c89ce0faf67228777ab6e3896a3cc0c04bfb04..d55798096640ba107277033d879e18d600aad0b8 100644 (file)
@@ -84,7 +84,6 @@ static void
 teardown (void *path)
 {
        p11_token_free (test.token);
-       memset (&test, 0, sizeof (test));
 }
 
 static void
@@ -93,6 +92,7 @@ teardown_temp (void *unused)
        p11_test_directory_delete (test.directory);
        teardown (test.directory);
        free (test.directory);
+       memset (&test, 0, sizeof (test));
 }
 
 static void