Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: UC_HOOK_MEM_READ only triggered once on Arm64 Host machine. #1935

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

saicao
Copy link

@saicao saicao commented Apr 12, 2024

Fix UC_HOOK_MEM_READ only triggered once on Arm64 Host machine.
add simple test for that.
#1908.

@saicao
Copy link
Author

saicao commented Apr 12, 2024

ppc64le host have same problem with mem hook.
It fail my unit tests. want to me remove the unit test I added?

@wtdcode
Copy link
Member

wtdcode commented Apr 12, 2024

ppc64le host have same problem with mem hook. It fail my unit tests. want to me remove the unit test I added?

You can disable tests for ppc64 by using defines as I did somethere.

@Gavin0210
Copy link

I try it in android arm64,but not work

@droe
Copy link

droe commented Sep 6, 2024

fwiw, this patch, applied on top of d4b9248, addresses this (hugely annoying) issue for me.

What is needed to get a fix merged?

@glennsec glennsec mentioned this pull request Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants