diff options
| author | raylu-hdf <60487644+raylu-hdf@users.noreply.github.com> | 2022-08-05 16:05:07 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-08-05 16:05:07 (GMT) |
| commit | 54f116b42db2d2aabd6fdb58cebb99f04f106310 (patch) | |
| tree | 08509670b3bb1494e27266412701397c5fea2353 /src/H5Tcommit.c | |
| parent | bf07e0f2c9b381509abbde59fca8bea5445da261 (diff) | |
| download | hdf5-54f116b42db2d2aabd6fdb58cebb99f04f106310.zip hdf5-54f116b42db2d2aabd6fdb58cebb99f04f106310.tar.gz hdf5-54f116b42db2d2aabd6fdb58cebb99f04f106310.tar.bz2 | |
Sanity checks for H5Xcreate_anon functions to make sure the IDs passed in are valid (#1967)
* Sanity checks for H5Xcreate_anon functions to make sure the IDs passed in are valid.
* Removed unnecessary sanity checks.
Co-authored-by: songyulu <songyulu@jelly.ad.hdfgroup.org>
Diffstat (limited to 'src/H5Tcommit.c')
| -rw-r--r-- | src/H5Tcommit.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/H5Tcommit.c b/src/H5Tcommit.c index b49271f..47468fb 100644 --- a/src/H5Tcommit.c +++ b/src/H5Tcommit.c @@ -349,6 +349,11 @@ H5Tcommit_anon(hid_t loc_id, hid_t type_id, hid_t tcpl_id, hid_t tapl_id) else if (TRUE != H5P_isa_class(tcpl_id, H5P_DATATYPE_CREATE)) HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not datatype creation property list") + if (H5P_DEFAULT == tapl_id) + tapl_id = H5P_DATATYPE_ACCESS_DEFAULT; + else if (TRUE != H5P_isa_class(tapl_id, H5P_DATATYPE_ACCESS)) + HGOTO_ERROR(H5E_ARGS, H5E_BADTYPE, FAIL, "not datatype access property list") + /* Verify access property list and set up collective metadata if appropriate */ if (H5CX_set_apl(&tapl_id, H5P_CLS_TACC, loc_id, TRUE) < 0) HGOTO_ERROR(H5E_DATATYPE, H5E_CANTSET, FAIL, "can't set access property list info") |