diff options
author | Dana Robinson <43805+derobins@users.noreply.github.com> | 2021-04-29 11:57:02 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-04-29 11:57:02 (GMT) |
commit | 138bc52facad0e6be4cfd13a860bb628c1dfd626 (patch) | |
tree | 8fc9c29e5cfbbbf05293f3a0c34d65e55fe82bcd /test/testerror.sh.in | |
parent | 00dc456cec2a6820bbb05d8a9a2f1967bf30f7e8 (diff) | |
download | hdf5-138bc52facad0e6be4cfd13a860bb628c1dfd626.zip hdf5-138bc52facad0e6be4cfd13a860bb628c1dfd626.tar.gz hdf5-138bc52facad0e6be4cfd13a860bb628c1dfd626.tar.bz2 |
Fix for a segfault when H5Pset_fapl_log is passed an invalid fapl ID (#607)
* Committing clang-format changes
* Fixes an issue where H5Pset_fapl_log sefaults when passed an invalid
fapl ID
This was due to a pointer-containing struct being memset after the first
internal API call. If the first call failed, the error condition would
check if the pointer was not NULL and then attempt to free it if not.
This would lead to the freeing of a wild pointer if an invalid fapl ID
were passed in.
This was fixed by reordering the memset and adding a test to ensure the
problem stays fixed.
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Diffstat (limited to 'test/testerror.sh.in')
0 files changed, 0 insertions, 0 deletions