Skip to content

Commit

Permalink
Fix loading plugin fails with missing directory GH issue HDFGroup#3248
Browse files Browse the repository at this point in the history
  • Loading branch information
byrnHDF committed Aug 1, 2023
1 parent 0ff29e7 commit 1ed7058
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
9 changes: 8 additions & 1 deletion release_docs/RELEASE.txt
Original file line number Diff line number Diff line change
Expand Up @@ -169,9 +169,16 @@ New Features

Library:
--------
- Change the error handling for a not found path in the find plugin process.

While attempting to load a plugin the HDF5 library will fail if one of the
directories in the plugin paths does not exist, even if there are more paths
to check. Instead of exiting the function with an error, just logged the error
and continue processing the list of paths to check.

- Implemented support for temporary security credentials for the Read-Only
S3 (ROS3) file driver.

When using temporary security credentials, one also needs to specify a
session/security token next to the access key id and secret access key.
This token can be specified by the new API function H5Pset_fapl_ros3_token().
Expand Down
2 changes: 1 addition & 1 deletion src/H5PLpath.c
Original file line number Diff line number Diff line change
Expand Up @@ -801,7 +801,7 @@ H5PL__find_plugin_in_path_table(const H5PL_search_params_t *search_params, hbool

/* Search for the plugin in this path */
if (H5PL__find_plugin_in_path(search_params, found, H5PL_paths_g[u], plugin_info) < 0)
HGOTO_ERROR(H5E_PLUGIN, H5E_CANTGET, FAIL, "search in path %s encountered an error",
HDONE_ERROR(H5E_PLUGIN, H5E_CANTGET, FAIL, "search in path %s encountered an error",
H5PL_paths_g[u])

/* Break out if found */
Expand Down

0 comments on commit 1ed7058

Please sign in to comment.