summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJake Smith <jake.smith@hdfgroup.org>2020-06-15 19:36:41 (GMT)
committerJake Smith <jake.smith@hdfgroup.org>2020-06-15 19:36:41 (GMT)
commitb718bf567f5ccd649a4d3153276f47a56ee0d939 (patch)
tree8cd525de864017959df1c85681287e8fd7145425
parent70011fbfcbeb15b2ce9c1784f3f61d4583c4c259 (diff)
parent6462c67b6578e48c1ef6d847be59c0b5e3598a50 (diff)
downloadhdf5-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.c3
-rw-r--r--test/vfd.c3
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 */
diff --git a/test/vfd.c b/test/vfd.c
index 97995d2..d94aec6 100644
--- a/test/vfd.c
+++ b/test/vfd.c
@@ -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);