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

Use ELF Header to check output file #43

Closed
wants to merge 1 commit into from
Closed

Conversation

tiann
Copy link

@tiann tiann commented Jan 4, 2022

This should also fix #35

@bbqsrc
Copy link
Owner

bbqsrc commented Feb 9, 2022

I will not be supporting binaries in cargo-ndk at this time. It is not a use case that Android generally supports, and I do not want to add code complexity to maintain it.

@bbqsrc bbqsrc closed this Feb 9, 2022
@tiann
Copy link
Author

tiann commented Feb 9, 2022

The reason is that this project use libXXXX.so to recognize library files, this it not so accurate. The ELF format is standard, if using ELF MAGIC number to recognize library files, it can also recognize executable binaries. it works well.

@tiann
Copy link
Author

tiann commented Feb 9, 2022

And also, using ndk-build to build executable binaries is generally used(Such as exploit PoCs, native servers), and is supported by Android ndk toolchains

@zchrissirhcz
Copy link

Android Native Development is a very import part of Android development. Compiling to binaries should be supported.

Repository owner locked and limited conversation to collaborators Mar 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

executable file is not striped in release build
3 participants