Skip to content

Adding function to get rich header hash#331

Closed
ForensicITGuy wants to merge 5 commits intoerocarrera:masterfrom
ForensicITGuy:add-rich-header-hash
Closed

Adding function to get rich header hash#331
ForensicITGuy wants to merge 5 commits intoerocarrera:masterfrom
ForensicITGuy:add-rich-header-hash

Conversation

@ForensicITGuy
Copy link
Copy Markdown
Contributor

Added the function get_rich_header_hash() which returns the md5 hash of the PE Rich Header, which can then be used with VT or other searches

@erocarrera
Copy link
Copy Markdown
Owner

Hi, I see how this may come handy, thanks!
Given that md5 is on the way out and that pefile already imports sha1/sha256/sha512 in addition to md5 I think it'd be great if all were supported. Maybe you could add a family of methods like get_rich_header_md5() and likewise for the other supported hashes? alternatively get_rich_header_hash() could default to md5 but take an optional argument specifying the hash algorithm.

@ForensicITGuy
Copy link
Copy Markdown
Contributor Author

Hey @erocarrera that's an awesome idea, I hadn't even thought of it!

I'll get working on that in the next couple of days and should have something on it soon

@ForensicITGuy
Copy link
Copy Markdown
Contributor Author

Gotta close this PR b/c I somehow messed up my branch, will open another

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