Skip to content

Conversation

mattjala
Copy link
Contributor

@mattjala mattjala commented Oct 3, 2023

Fixes a segfault that would occur when using a non-native VOL with an H5Iiterate callback that doesn't check for NULL itself (for example, H5Z__check_unregister_dset_cb as used in H5Zunregister).

Fixes a segfault that would occur when using a non-native VOL
while using an H5Iiterate callback that doesn't check for NULL
itself (for example, H5Z__check_unregister_dset_cb as used
in H5Zunregister).
@mattjala mattjala added Merge - To 1.14 Component - C Library Core C library issues (usually in the src directory) labels Oct 3, 2023
@mattjala mattjala marked this pull request as draft October 3, 2023 18:40
@mattjala
Copy link
Contributor Author

mattjala commented Oct 3, 2023

This ended up causing more problem for the VOL connectors than it solved - a lot of API calls use H5Iiterate in a way that doesn't depend on the provided object pointer. A different workaround for the problem this was trying to solve (using H5Zunregister with VOL connectors) is provided in #3629.

@mattjala mattjala closed this Oct 3, 2023
@mattjala mattjala deleted the id_iterate_null_fail branch September 20, 2024 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - C Library Core C library issues (usually in the src directory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant