From d7bfb441791196caaf16b0ae7e9eaab449e491e7 Mon Sep 17 00:00:00 2001 From: Jordan Henderson Date: Wed, 23 Aug 2023 20:34:30 -0500 Subject: [PATCH] Avoid H5Ocopy in h5repack for variable-length string types --- tools/src/h5repack/h5repack_copy.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tools/src/h5repack/h5repack_copy.c b/tools/src/h5repack/h5repack_copy.c index 81b33c3093f..2d0819b1fa3 100644 --- a/tools/src/h5repack/h5repack_copy.c +++ b/tools/src/h5repack/h5repack_copy.c @@ -810,6 +810,10 @@ do_copy_objects(hid_t fidin, hid_t fidout, trav_table_t *travt, pack_opt_t *opti /* early detection of variable-length types */ if ((is_vlen = H5Tdetect_class(ftype_id, H5T_VLEN)) < 0) H5TOOLS_GOTO_ERROR((-1), "H5Tdetect_class failed"); + if (!is_vlen) { + if ((is_vlen = H5Tis_variable_str(ftype_id)) < 0) + H5TOOLS_GOTO_ERROR((-1), "H5Tis_variable_str failed"); + } /* Check if the datatype is committed */ if ((is_named = H5Tcommitted(ftype_id)) < 0)