Skip to content

Conversation

@xal-0
Copy link
Member

@xal-0 xal-0 commented Jul 29, 2025

Use ios_t as a string builder for library paths to support really long DL_LOAD_PATH entries. jl_load_dynamic_library can't use the Julia GC to allocate because we're called from JuliaOJIT::DLSymOptimizer::lookup with symbols_mutex held.

Fixes #59130.

@xal-0 xal-0 added the bugfix This change fixes an existing bug label Jul 29, 2025
@xal-0 xal-0 changed the title Use ios to supported ridiculous DL_LOAD_PATH length Use ios to support ridiculous DL_LOAD_PATH length Jul 29, 2025
@xal-0 xal-0 merged commit ebf55c0 into JuliaLang:master Jul 30, 2025
10 checks passed
@xal-0 xal-0 deleted the fix-load-dynamic-library-buf branch July 30, 2025 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This change fixes an existing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Intermittent CI segfault in gethostname

2 participants