summaryrefslogtreecommitdiffstats
path: root/tools/lib
diff options
context:
space:
mode:
authorJonathan Kim <jkm@hdfgroup.org>2011-03-18 20:13:16 (GMT)
committerJonathan Kim <jkm@hdfgroup.org>2011-03-18 20:13:16 (GMT)
commitf845e716bc34a5929e979cf046ab3d032c7e21d8 (patch)
treecc20ed94dc7b3d8e207480cded5a88ee52086fa5 /tools/lib
parentd697acbfb588b0406403109398105e12562e2d3e (diff)
downloadhdf5-f845e716bc34a5929e979cf046ab3d032c7e21d8.zip
hdf5-f845e716bc34a5929e979cf046ab3d032c7e21d8.tar.gz
hdf5-f845e716bc34a5929e979cf046ab3d032c7e21d8.tar.bz2
[svn-r20274] Purpose:
Fixed Bug 2214 - GMQS: h5diff - memory leak in diff() Description: Memory leaks occurred when accessing symbolic links with --follow-symlink option. Test cases are in testing script. (400 - 456, 501 - 518) An example test : valgrind --leak-check=full h5diff --follow-symlinks -v h5diff_softlinks.h5 h5diff_softlinks.h5 Tested: jam (linux32-LE), amani (linux64-LE), heiwa (linuxppc64-BE), tejeda (mac32-LE), linew (solaris-BE), Cmake - jam
Diffstat (limited to 'tools/lib')
-rw-r--r--tools/lib/h5diff.c13
1 files changed, 6 insertions, 7 deletions
diff --git a/tools/lib/h5diff.c b/tools/lib/h5diff.c
index 5f406cf..a616dba 100644
--- a/tools/lib/h5diff.c
+++ b/tools/lib/h5diff.c
@@ -1958,9 +1958,6 @@ hsize_t diff(hid_t file1_id,
if(options->m_verbose)
print_found(nfound);
- /* free link info buffer */
- HDfree(linkinfo1.trg_path);
- HDfree(linkinfo2.trg_path);
}
break;
@@ -2007,10 +2004,6 @@ hsize_t diff(hid_t file1_id,
file2_id, path2,
options, linkinfo1.trg_type);
}
-
- /* free link info buffer */
- HDfree(linkinfo1.trg_path);
- HDfree(linkinfo2.trg_path);
} /* end if */
else
{
@@ -2045,6 +2038,12 @@ hsize_t diff(hid_t file1_id,
break;
}
+ /* free link info buffer */
+ if (linkinfo1.trg_path)
+ HDfree(linkinfo1.trg_path);
+ if (linkinfo2.trg_path)
+ HDfree(linkinfo2.trg_path);
+
return nfound;
out: