Skip to content

Expose and use get_pefile_obj rather than having local copies#1585

Merged
ikelos merged 3 commits intodevelopfrom
issues/issue1577
Jan 31, 2025
Merged

Expose and use get_pefile_obj rather than having local copies#1585
ikelos merged 3 commits intodevelopfrom
issues/issue1577

Conversation

@ikelos
Copy link
Member

@ikelos ikelos commented Jan 27, 2025

Fixes #1577

@ikelos ikelos requested a review from atcuno January 27, 2025 23:14
@ikelos ikelos changed the title Issues/issue1577 Expose and use get_pefile_obj rather than having local copies Jan 30, 2025
@atcuno
Copy link
Contributor

atcuno commented Jan 31, 2025

This is good to go ikelos. I tested on it samples with skeleton key's present and it still finds them

@ikelos ikelos merged commit 1f983ac into develop Jan 31, 2025
24 checks passed
@ikelos ikelos deleted the issues/issue1577 branch January 31, 2025 17:30
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.

skeleton_key_check._get_pefile_obj looks very similar to pe_symbols._get_pefile_obj

2 participants