[test/unicode] Fix double-free
authorBehdad Esfahbod <behdad@behdad.org>
Fri, 27 May 2011 06:46:28 +0000 (02:46 -0400)
committerBehdad Esfahbod <behdad@behdad.org>
Fri, 27 May 2011 07:40:04 +0000 (03:40 -0400)
Caught by "make check-valgrind".

test/Makefile.am
test/test-unicode.c

index 4d6c460..46b21be 100644 (file)
@@ -102,7 +102,7 @@ valgrind_verbose = $(valgrind_verbose_$(V))
 valgrind_verbose_ = $(valgrind_verbose_$(AM_DEFAULT_VERBOSITY))
 valgrind_verbose_0 = | \
        grep '\(^[^=]\|ERROR SUMMARY\|definitely lost\|indirectly lost\)' | grep -v ': 0'
-
+# TODO: The following check does not fail if valgrind finds error.  It should.
 check-valgrind:
        $(AM_V_at)$(MAKE) $(AM_MAKEFLGS) check-tool TOOL="valgrind $(VALGRIND_FLAGS)" \
        2>&1 | tee valgrind-log $(valgrind_verbose)
index e163aba..cce4fd2 100644 (file)
@@ -633,8 +633,6 @@ test_unicode_setters (void)
     g_assert (data[0].freed && !data[1].freed);
     hb_unicode_funcs_destroy (uf);
     g_assert (data[0].freed && !data[1].freed);
-
-    hb_unicode_funcs_destroy (uf);
   }
 }