From 0e482ec6ba30b613f2d7ee97c61be458c5aebcd6 Mon Sep 17 00:00:00 2001 From: Behdad Esfahbod Date: Fri, 27 May 2011 02:46:28 -0400 Subject: [PATCH] [test/unicode] Fix double-free Caught by "make check-valgrind". --- test/Makefile.am | 2 +- test/test-unicode.c | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/test/Makefile.am b/test/Makefile.am index 4d6c46063..46b21be68 100644 --- a/test/Makefile.am +++ b/test/Makefile.am @@ -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) diff --git a/test/test-unicode.c b/test/test-unicode.c index e163aba3c..cce4fd2d5 100644 --- a/test/test-unicode.c +++ b/test/test-unicode.c @@ -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); } }