summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAllen Byrne <byrn@hdfgroup.org>2018-10-16 18:10:48 (GMT)
committerAllen Byrne <byrn@hdfgroup.org>2018-10-16 18:10:48 (GMT)
commitc39449fa12fb45f12d97cce554c9f023645ff379 (patch)
tree2dd47bd4278276bd7eeec72543c3a0a863e67fe2
parent6b65a1b78f13adaa1cc95a805c558ac4f8e98278 (diff)
parent1e8ef703cdc057211b3587be776a6e9f62e8f76f (diff)
downloadhdf5-c39449fa12fb45f12d97cce554c9f023645ff379.zip
hdf5-c39449fa12fb45f12d97cce554c9f023645ff379.tar.gz
hdf5-c39449fa12fb45f12d97cce554c9f023645ff379.tar.bz2
Merging in latest from upstream (HDFFV/hdf5:refs/heads/develop)
* commit '1e8ef703cdc057211b3587be776a6e9f62e8f76f': Fixed a C++ style commenting issue and removed an unused field from H5F_trav_obj_ids_t. Fixed a memory issue in H5Drefresh() where the dataset's H5VL_object_t gets closed but we try to use it to find the driver.
-rw-r--r--src/H5F.c2
-rw-r--r--src/H5Fpkg.h1
-rw-r--r--src/H5Oflush.c13
3 files changed, 9 insertions, 7 deletions
diff --git a/src/H5F.c b/src/H5F.c
index 9258830..2dcafb9 100644
--- a/src/H5F.c
+++ b/src/H5F.c
@@ -477,8 +477,6 @@ H5Fget_obj_ids(hid_t file_id, unsigned types, size_t max_objs, hid_t *oid_list)
else {
H5F_trav_obj_ids_t udata;
- /* XXX (VOL MERGE): Unclear why this is commented out */
- //udata.types = types | H5F_OBJ_LOCAL;
udata.max_objs = max_objs;
udata.oid_list = oid_list;
udata.obj_count = &ret_value;
diff --git a/src/H5Fpkg.h b/src/H5Fpkg.h
index f8ab5a9..e97144f 100644
--- a/src/H5Fpkg.h
+++ b/src/H5Fpkg.h
@@ -418,7 +418,6 @@ typedef struct {
size_t max_objs;
hid_t *oid_list;
ssize_t *obj_count; /* number of objects counted so far */
- unsigned types; /* types of objects to be counted */
} H5F_trav_obj_ids_t;
/*****************************/
diff --git a/src/H5Oflush.c b/src/H5Oflush.c
index b27f01b..84095d9 100644
--- a/src/H5Oflush.c
+++ b/src/H5Oflush.c
@@ -301,6 +301,7 @@ H5O_refresh_metadata(hid_t oid, H5O_loc_t oloc)
H5O_loc_t obj_oloc;
H5G_name_t obj_path;
H5O_shared_t cached_H5O_shared;
+ H5VL_t *driver = NULL;
/* Create empty object location */
obj_loc.oloc = &obj_oloc;
@@ -318,25 +319,29 @@ H5O_refresh_metadata(hid_t oid, H5O_loc_t oloc)
if(H5T_save_refresh_state(oid, &cached_H5O_shared) < 0)
HGOTO_ERROR(H5E_DATATYPE, H5E_CANTOPENOBJ, FAIL, "unable to save datatype state")
- /* Get the VOL object from the ID */
+ /* Get the VOL object from the ID and cache a pointer to the driver.
+ * The vol_obj will disappear when the underlying object is closed, so
+ * we can't use that directly.
+ */
if(NULL == (vol_obj = H5VL_get_object(oid)))
HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "invalid object identifier")
+ driver = vol_obj->driver;
/* Bump the number of references on the VOL driver.
* If you don't do this, VDS refreshes can accidentally close the driver.
*/
- vol_obj->driver->nrefs++;
+ driver->nrefs++;
/* Close object & evict its metadata */
if((H5O__refresh_metadata_close(oid, oloc, &obj_loc)) < 0)
HGOTO_ERROR(H5E_OHDR, H5E_CANTLOAD, FAIL, "unable to refresh object")
/* Re-open the object, re-fetching its metadata */
- if((H5O_refresh_metadata_reopen(oid, &obj_loc, vol_obj->driver, FALSE)) < 0)
+ if((H5O_refresh_metadata_reopen(oid, &obj_loc, driver, FALSE)) < 0)
HGOTO_ERROR(H5E_OHDR, H5E_CANTLOAD, FAIL, "unable to refresh object")
/* Restore the number of references on the VOL driver */
- vol_obj->driver->nrefs--;
+ driver->nrefs--;
/* Restore important datatype state */
if(H5I_get_type(oid) == H5I_DATATYPE)