DynamoRIO / DynamoRIO/dynamorio

switch drconfiglib and drinjectlib to be static instead of shared

Open
#1,737 3 comments 0 reactions 0 assignees View on GitHub
Component-Build Component-Extension Type-Feature
Dominant language
C
Stars
3.2k
Forks
629
Avg merge
2d 15h
Merged PRs (30d)
31

Description

The loader on Android (xref #1701) does not support DT_RUNPATH. We could try to swap in the older DT_RPATH, or we could switch drconfiglib and drinjectlib to be static instead of shared.

If we change drconfiglib and drinjectlib to be static, old frontends might break, so better to have two different libs (shared + static) like DRMF does? OTOH, these libs are BSD unlike DRMF, and changing will only break someone who built their own frontend but doesn't ship their own DR, which seems unlikely.

Much simpler to be static: avoid all the add_rel_rpaths() and the Mac #1375 INSTALL_NAME_DIR stuff, and all the copying of libs on Windows. drfrontendlib and drdecodelib are already static.

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.