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

add support for spc link / urls in spc indirect data #1153

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

bhoehl
Copy link

@bhoehl bhoehl commented Jan 7, 2025

@romainthomas
My use case of LIEF is reading signatures and validating them.
This is not necessarily using executables, instead I am validating fonts.
The Spc Indirect Data can contain other values instead of files, also urls.
This PR adresses the issue to support urls too.
If possible please review and merge, as I would really like to continue using this lib.
There is no real support for PKCS#9 for python out there ...

Thanks
Bernd

romainthomas and others added 3 commits January 5, 2025 17:45
As demonstrated by @lebr0nli in the sample `ELF/issue_dynamic_table.elf`,
the PT_DYNAMIC segment can have an "invalid" offset while still running
correctly.
This commit changes the logic of the dynamic table processing by using
the PT_LOAD segment that wraps this table
@romainthomas
Copy link
Member

Interesting use case :) Would you mind sharing a sample of a font using this PKCS #9 signature?

@bhoehl
Copy link
Author

bhoehl commented Jan 13, 2025

@romainthomas due to legal rights I am not able to do that
but, if you own a Windows system, any arial font inside has a digital signature ( DSIG table ).
\Windows\fonts\arial.ttf

@bhoehl
Copy link
Author

bhoehl commented Jan 13, 2025

@romainthomas
an example of a such a signature
I am loading this file using:

LIEF::logging::set_level(LIEF::logging::LEVEL::DEBUG);
auto sig = SignatureParser::parse(data, false);

signature.zip

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.

2 participants