diff options
author | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2020-08-18 05:49:09 (GMT) |
---|---|---|
committer | Binh-Minh Ribler <bmribler@hdfgroup.org> | 2020-08-18 05:49:09 (GMT) |
commit | ede1b9b1dd53caad6dfce8ec7bc4aa7b3dbce62c (patch) | |
tree | 5c7815e4f0dea82e308ef638f4846bdf2f0867f6 | |
parent | fa7ae0e001289e8cc2c04c2f3cf8ff03bfc83ddb (diff) | |
download | hdf5-ede1b9b1dd53caad6dfce8ec7bc4aa7b3dbce62c.zip hdf5-ede1b9b1dd53caad6dfce8ec7bc4aa7b3dbce62c.tar.gz hdf5-ede1b9b1dd53caad6dfce8ec7bc4aa7b3dbce62c.tar.bz2 |
Fix HDFFV-11101
Description
Added initialization to local structs in the src function
H5MF_settle_raw_data_fsm() and the test function test_bt2_hdr_fd()
to prevent the following error in two different occurrences:
MemorySanitizer: use-of-uninitialized-value
Platforms tested:
Linux/64 (jelly)
-rw-r--r-- | src/H5MF.c | 4 | ||||
-rw-r--r-- | test/dsets.c | 3 |
2 files changed, 7 insertions, 0 deletions
@@ -2674,6 +2674,10 @@ H5MF_settle_raw_data_fsm(H5F_t *f, hbool_t *fsm_settled) HDassert(f->shared); HDassert(fsm_settled); + /* Initialize structs */ + HDmemset(&fsinfo, 0, sizeof(fsinfo)); + HDmemset(&fs_stat, 0, sizeof(fs_stat)); + /* Only need to settle things if we are persisting the free space info * and allocation/deallocation has occurred. */ diff --git a/test/dsets.c b/test/dsets.c index d72c65b..f7aeb1a 100644 --- a/test/dsets.c +++ b/test/dsets.c @@ -11231,6 +11231,9 @@ test_bt2_hdr_fd(const char *env_h5_driver, hid_t fapl) TESTING("Version 2 B-tree chunk index header flush dependencies handled correctly"); + /* Initialize struct */ + HDmemset(&info, 0, sizeof(info)); + /* Skip this test if SWMR I/O is not supported for the VFD specified * by the environment variable. */ |