DynamoRIO / DynamoRIO/dynamorio

update native_exec to properly swap the signal mask, TLS, etc.

Open
#2,375 0 comments 0 reactions 0 assignees View on GitHub
Bug-AppFail OpSys-Linux
Dominant language
C
Stars
3.2k
Forks
629
Avg merge
2d 15h
Merged PRs (30d)
31

Description

The native_exec code is not using os_thread_{,not_}under_dynamo() which
means it is not using the fixes and changes from the past year or so:
swapping the signal mask (#2337), clearing the TLS (#2089), etc.

It looks like adding the os_thread_{,not_}under_dynamo() calls and the
os_thread_re_take_over() to get the TLS and dcontext back are
straightforward for default options, so I will go ahead and do it, but
complex for -native_exec_opt as they need to be in generated code.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.