diff options
| author | vchoi-hdfgroup <55293060+vchoi-hdfgroup@users.noreply.github.com> | 2023-04-12 04:06:29 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-04-12 04:06:29 (GMT) |
| commit | 67d4b5eaf8cbfba8d2aa6e5309c5ce7807156a1d (patch) | |
| tree | 63f1ba18a9424005ba5eb0a0cfa509ea7538e603 /release_docs | |
| parent | d1727fe8f3c5fcbe8679f8c3e1cc3b98cc6ca99e (diff) | |
| download | hdf5-67d4b5eaf8cbfba8d2aa6e5309c5ce7807156a1d.zip hdf5-67d4b5eaf8cbfba8d2aa6e5309c5ce7807156a1d.tar.gz hdf5-67d4b5eaf8cbfba8d2aa6e5309c5ce7807156a1d.tar.bz2 | |
Fix for github issue #2599: (#2665)
* Fix for github issue #2599:
As indicated in the description, memory leak is detected when running "./h5dump pov".
The problem is: when calling H5O__add_cont_msg() from H5O__chunk_deserialize(),
memory is allocated for cont_msg_info->msgs. Eventually, when the library tries to load
the continuation message via H5AC_protect() in H5O_protect(), error is
encountered due to illegal info in the continuation message.
Due to the error, H5O_protect() exits but the memory allocated for cont_msg_info->msgs is not freed.
When we figure out how to handle fuzzed files that we didn't generate,
a test needs to be added to run h5dump with the provided "pov" file.
* Add message to release notes for the fix to github issue #2599.
Diffstat (limited to 'release_docs')
| -rw-r--r-- | release_docs/RELEASE.txt | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/release_docs/RELEASE.txt b/release_docs/RELEASE.txt index c04ead5..0fde1ab 100644 --- a/release_docs/RELEASE.txt +++ b/release_docs/RELEASE.txt @@ -151,6 +151,21 @@ Bug Fixes since HDF5-1.13.3 release =================================== Library ------- + + - Memory leak + + Memory leak was detected when running h5dump with "pov". The memory was allocated + via H5FL__malloc() in hdf5/src/H5FL.c + + The fuzzed file "pov" was an HDF5 file containing an illegal continuation message. + When deserializing the object header chunks for the file, memory is allocated for the + array of continuation messages (cont_msg_info->msgs) in continuation message info struct. + As error is encountered in loading the illegal message, the memory allocated for + cont_msg_info->msgs needs to be freed. + + (VC - 2023/04/11 GH-2599) + + - Fixed memory leaks that could occur when reading a dataset from a malformed file |