diff options
| author | Quincey Koziol <koziol@hdfgroup.org> | 2015-09-25 16:29:02 (GMT) |
|---|---|---|
| committer | Quincey Koziol <koziol@hdfgroup.org> | 2015-09-25 16:29:02 (GMT) |
| commit | 10837a2a28d0fd2a1ece51f89af286bafd20204a (patch) | |
| tree | a204af932ccec79b7c686305f1dec45d602bf4c8 | |
| parent | 718b311c110fa5895c0daccb6826525662de6d2c (diff) | |
| download | hdf5-10837a2a28d0fd2a1ece51f89af286bafd20204a.zip hdf5-10837a2a28d0fd2a1ece51f89af286bafd20204a.tar.gz hdf5-10837a2a28d0fd2a1ece51f89af286bafd20204a.tar.bz2 | |
[svn-r27876] Description:
Make virtual layout return to the property list occur in all cases, even
on failure.
Tested on:
MacOSX/64 10.10.5 (amazon) w/serial & parallel
(h5committest not needed on this branch)
| -rw-r--r-- | src/H5Pdcpl.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/H5Pdcpl.c b/src/H5Pdcpl.c index 5a0e32f..b1ddff0 100644 --- a/src/H5Pdcpl.c +++ b/src/H5Pdcpl.c @@ -2251,14 +2251,15 @@ H5Pset_virtual(hid_t dcpl_id, hid_t vspace_id, const char *src_file_name, /* Finish adding entry */ virtual_layout.storage.u.virt.list_nused++; +done: /* Set VDS layout information in property list */ + /* (Even on faliure, so there's not a mangled layout struct in the list) */ if(H5P_poke(plist, H5D_CRT_LAYOUT_NAME, &virtual_layout) < 0) { HGOTO_ERROR(H5E_PLIST, H5E_CANTSET, FAIL, "can't set layout") if(old_list != virtual_layout.storage.u.virt.list) free_list = TRUE; } /* end if */ -done: /* Check if the entry has been partly allocated but not added to the * property list or not included in list_nused */ if(ret_value < 0) { |