Skip to content

Implements load libcapstone.so with RTLD_DEEPBIND #2912#2913

Merged
Rot127 merged 4 commits into
capstone-engine:nextfrom
AGhebrea:python_rtld_deepbind
May 10, 2026
Merged

Implements load libcapstone.so with RTLD_DEEPBIND #2912#2913
Rot127 merged 4 commits into
capstone-engine:nextfrom
AGhebrea:python_rtld_deepbind

Conversation

@AGhebrea
Copy link
Copy Markdown
Contributor

@AGhebrea AGhebrea commented May 8, 2026

Your checklist for this pull request

  • I've documented or updated the documentation of every API function and struct this PR changes.
  • I've added tests that prove my fix is effective or that my feature works (if possible)

Detailed description

A possible implementation feature request #2912

Test plan

I locally installed the modified python bindings locally along with cstest_py and ran

cstest_py ./tests/details

All the tests passed. I think that this is sufficient, given that I don't modify the functionality of the capstone code.

@github-actions github-actions Bot added the Python Bindings label May 8, 2026
@Rot127
Copy link
Copy Markdown
Collaborator

Rot127 commented May 8, 2026

Now we have two version :D

#2913

Comment thread bindings/python/capstone/__init__.py Outdated
@AGhebrea
Copy link
Copy Markdown
Contributor Author

Tests were failing because musl does not implement RTLD_DEEPBIND.
mode = getattr(os, 'RTLD_DEEPBIND', 0) in 2fd8ea3 fixes that.

@Rot127 Rot127 mentioned this pull request May 10, 2026
2 tasks
@Rot127 Rot127 merged commit e46f64f into capstone-engine:next May 10, 2026
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Python Bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants