Skip to content

Reinstate the kernel32.dll exphash test - #647

Merged
j-t-1 merged 3 commits into
erocarrera:masterfrom
HuzaifaChaudary:tests/exphash-kernel32
Nov 8, 2026
Merged

j-t-1 merged 3 commits into
erocarrera:masterfrom
HuzaifaChaudary:tests/exphash-kernel32

Conversation

@HuzaifaChaudary

Copy link
Copy Markdown
Contributor

part of #584 .

ce0e9801... was computed on a different kernel32.dll from the one in pefile-tests . the sha256 values from before 0c747a7 still match mfc40.dll and the 66c74e4c sample , so those are the same files , but neither the sha256 nor the md5 value matches this kernel32.dll .

radare2 ( rabin2 -Ej ) lists the same 954 exports in the same order as pefile , and the md5 of the lowercase names joined with , is 247dcd1aa98526819ecf3c09cb12ad83 , the value get_exphash() returns .

the kernel32.dll in pefile-tests is a different build from the one ce0e9801 was taken from . the old sha256 values from 2022 still match mfc40.dll and the 66c74e4c sample , but neither the old sha256 nor the md5 matches this kernel32.dll .

radare2 ( rabin2 -E ) lists the same 954 exports in the same order , and the lowercase , comma joined names give 247dcd1aa98526819ecf3c09cb12ad83 , the same as get_exphash().
Copilot AI balanced review requested due to automatic review settings November 3, 2026 13:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

Comment thread tests/pefile_test.py Resolved
@j-t-1
j-t-1 merged commit bff4b59 into erocarrera:master Nov 8, 2026
25 checks passed
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