Skip to content

Port hooking_libc example to C++#1366

Merged
cnheitman merged 5 commits intoJonathanSalwan:dev-v1.0from
selimsandal:master
Jul 22, 2025
Merged

Port hooking_libc example to C++#1366
cnheitman merged 5 commits intoJonathanSalwan:dev-v1.0from
selimsandal:master

Conversation

@selimsandal
Copy link
Contributor

I ported the hooking_libc Python example to C++, ran sample_1 and some other programs successfully.

@JonathanSalwan, also I wrote homebrew formulas for Bitwuzla and Triton to use them with CMake easily. Can you check them? If they are okay, maybe we can send them to Homebrew.

Bitwuzla
Triton

@JonathanSalwan
Copy link
Owner

Yeah sure, it could be great to have Triton on homebrew :)

@JonathanSalwan JonathanSalwan added this to the v1.0 milestone Oct 16, 2024
@JonathanSalwan
Copy link
Owner

Can you add your example in this CMakeLists.txt. It's used to see if all examples still compile over releases.

add_executable(hooking_libc hooking_libc.cpp)
set_property(TARGET hooking_libc PROPERTY CXX_STANDARD 17)
target_link_libraries(hooking_libc triton)
add_test(TestHookingLibc hooking_libc)
add_dependencies(check hooking_libc)

@JonathanSalwan JonathanSalwan changed the base branch from master to dev-v1.0 October 17, 2024 19:09
@selimsandal
Copy link
Contributor Author

Can you add your example in this CMakeLists.txt. It's used to see if all examples still compile over releases.

add_executable(hooking_libc hooking_libc.cpp)
set_property(TARGET hooking_libc PROPERTY CXX_STANDARD 17)
target_link_libraries(hooking_libc triton)
add_test(TestHookingLibc hooking_libc)
add_dependencies(check hooking_libc)

Didn't see this, now added to cmake

@cnheitman
Copy link
Collaborator

Hi! This looks good. I was planning to rebase and merge. Any comments @JonathanSalwan ?

@JonathanSalwan
Copy link
Owner

@cnheitman yep. lgtm

@cnheitman cnheitman merged commit 654f91d into JonathanSalwan:dev-v1.0 Jul 22, 2025
6 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants