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.

Sign up for GitHub

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

Jump to bottom

Parsing of multiple signatures from PE files #380

Closed
metthal opened this issue Aug 23, 2018 · 4 comments
Closed

Parsing of multiple signatures from PE files #380

metthal opened this issue Aug 23, 2018 · 4 comments
Assignees
@HoundThe
Labels
C-fileformat C-fileinfo T-format-pe

Comments

@metthal
Copy link
Member

metthal commented Aug 23, 2018

We currently do not parse the signatures from PE files the same way as described here.

We do not support multiple WIN_CERTIFICATE structures and also we do not support parsing of SPC_NESTED_SIGNATURE.

We already have an implementation of simple ASN1 parser so it can be used for the second case (maybe after some modifications).
@wxsBSD
Copy link

wxsBSD commented Aug 23, 2018

I'm happy to provide samples that have multiple WIN_CERTIFICATE structures (they are EFI images signed by both MSFT and RHAT) if you don't have any.

@metthal
Copy link
Member Author

metthal commented Aug 23, 2018

I would really appreciate that. It would help a lot. Thank you.

@wxsBSD
Copy link

wxsBSD commented Aug 24, 2018

Sorry, didn’t see this response until just now. Send me an email and I’ll get you a zip of them!

@PeterMatula PeterMatula mentioned this issue Jul 23, 2020
Closed
@HoundThe HoundThe mentioned this issue Jan 28, 2021
Merged
6 tasks
@PeterMatula
Copy link
Collaborator

Solved in #902

PeterMatula added a commit that referenced this issue May 20, 2021
@PeterMatula
CHANGELOG.md: add entry for #902 and #380
b374920
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HoundThe HoundThe

Labels
C-fileformat C-fileinfo T-format-pe
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@metthal @wxsBSD @HoundThe @PeterMatula