diff options
author | Jake Smith <jake.smith@hdfgroup.org> | 2020-06-15 19:36:41 (GMT) |
---|---|---|
committer | Jake Smith <jake.smith@hdfgroup.org> | 2020-06-15 19:36:41 (GMT) |
commit | b718bf567f5ccd649a4d3153276f47a56ee0d939 (patch) | |
tree | 8cd525de864017959df1c85681287e8fd7145425 | |
parent | 70011fbfcbeb15b2ce9c1784f3f61d4583c4c259 (diff) | |
parent | 6462c67b6578e48c1ef6d847be59c0b5e3598a50 (diff) | |
download | hdf5-b718bf567f5ccd649a4d3153276f47a56ee0d939.zip hdf5-b718bf567f5ccd649a4d3153276f47a56ee0d939.tar.gz hdf5-b718bf567f5ccd649a4d3153276f47a56ee0d939.tar.bz2 |
Merge pull request #2615 in HDFFV/hdf5 from ~JAKE.SMITH/hdf5:bugfix/splitter_valgrind_patch to develop
* commit '6462c67b6578e48c1ef6d847be59c0b5e3598a50':
Patch a few holes in the Splitter VFD implementation.
-rw-r--r-- | src/H5FDsplitter.c | 3 | ||||
-rw-r--r-- | test/vfd.c | 3 |
2 files changed, 5 insertions, 1 deletions
diff --git a/src/H5FDsplitter.c b/src/H5FDsplitter.c index 13816a5..4ed3c4a 100644 --- a/src/H5FDsplitter.c +++ b/src/H5FDsplitter.c @@ -818,6 +818,9 @@ done: if (file_ptr->wo_file) { H5FD_close(file_ptr->wo_file); } + if (file_ptr->logfp) { + HDfclose(file_ptr->logfp); + } H5FL_FREE(H5FD_splitter_t, file_ptr); } } /* end if error */ @@ -2490,7 +2490,7 @@ driver_is_splitter_compatible(hid_t fapl_id) { H5FD_splitter_vfd_config_t vfd_config; hid_t split_fapl_id = H5I_INVALID_HID; - herr_t ret; + herr_t ret = SUCCEED; int ret_value = 0; split_fapl_id = H5Pcreate(H5P_FILE_ACCESS); @@ -2503,6 +2503,7 @@ driver_is_splitter_compatible(hid_t fapl_id) vfd_config.rw_fapl_id = H5P_DEFAULT; vfd_config.wo_fapl_id = fapl_id; HDstrncpy(vfd_config.wo_path, "nonesuch", H5FD_SPLITTER_PATH_MAX); + *vfd_config.log_file_path = '\0'; H5E_BEGIN_TRY { ret = H5Pset_fapl_splitter(split_fapl_id, &vfd_config); |