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

Search libc at user defined place, allow cross plattform analysis #206

Merged
merged 3 commits into from
Dec 15, 2022

Conversation

sreschke80
Copy link
Contributor

The libc is searched through the complete filesystem.
This can take a very long time and does not support analysis of cross plattform analysis.
For example one wants to analyse libraries for Android on a Linux PC.
The Linux PC uses glibc on x64 and Android another libc on ARM.

The optional parameter 'libcfile' allows to specify a libc.so file or a search path, where
libc should be found.
If libc is not found, the search is done through the complete filesystem.

…ilesystem

The parameter is optional and allows to specify a file or a path where libc is placed.
If libc is not found the search goes through the complete filesystem.
@Costinteo
Copy link

Costinteo commented Dec 4, 2022

Great pull request! Saved me a lot of time. Thanks!

Hopefully it gets merged.

Although, I think the idea of the script is to be built from the different pieces in src/.
So you might have to add these changes to src/core and the other functions you've changed. So that you will be able to then generate checksec with make build.

@sreschke80
Copy link
Contributor Author

sreschke80 commented Dec 5, 2022

Although, I think the idea of the script is to be built from the different pieces in src/. So you might have to add these changes to src/core and the other functions you've changed. So that you will be able to then generate checksec with make build.

I will try to integrate the changes into the src directory.

@slimm609
Copy link
Owner

sorry for the late response. Thanks for the contribution. Shellcheck is failing on several checks.

* adding quotes
* use -n instead of ! -z to check for non empty string
@sreschke80
Copy link
Contributor Author

sorry for the late response. Thanks for the contribution. Shellcheck is failing on several checks.

I fixed the issues with shellcheck.

@slimm609 slimm609 merged commit 41e291b into slimm609:main Dec 15, 2022
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.

3 participants