From a60f8c8f5ed705d281aeb70cf29d78371d3be996 Mon Sep 17 00:00:00 2001 From: Allen Byrne Date: Wed, 9 Mar 2011 08:57:41 -0500 Subject: [svn-r20208] Valgrind fix for memory leak in free_tables - 2nd malloc was freed by free_table calls but not the first malloc. Added HDfree('table') after each free_table() call. Tested: local linux and valgrind --- tools/h5dump/h5dump.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tools/h5dump/h5dump.c b/tools/h5dump/h5dump.c index 98bd6c9..81a6793 100644 --- a/tools/h5dump/h5dump.c +++ b/tools/h5dump/h5dump.c @@ -824,8 +824,11 @@ table_list_free(void) /* Free each table */ free_table(table_list.tables[u].group_table); + HDfree(table_list.tables[u].group_table); free_table(table_list.tables[u].dset_table); + HDfree(table_list.tables[u].dset_table); free_table(table_list.tables[u].type_table); + HDfree(table_list.tables[u].type_table); } /* Free the table list */ @@ -4478,6 +4481,7 @@ main(int argc, const char *argv[]) "xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\" " "xsi:schemaLocation=\"http://hdfgroup.org/DTDs/HDF5-File " "http://www.hdfgroup.org/DTDs/HDF5-File.xsd\">\n",xmlnsprefix,ns); + HDfree(ns); } } else { printf("\n", -- cgit v0.12