8000
Skip to content

Nicely handle non ELF files in checksec#2457

Merged
peace-maker merged 1 commit intoGallopsled:devfrom
tesuji:checksec-not-elf
Sep 11, 2024
Merged

Nicely handle non ELF files in checksec#2457
peace-maker merged 1 commit intoGallopsled:devfrom
tesuji:checksec-not-elf

Conversation

@tesuji
Copy link
Copy Markdown
Contributor
@tesuji tesuji commented Sep 10, 2024

Otherwise we have a traceback that contains 1/3 of a screen.

Traceback (most recent call last):
...
elftools.common.exceptions.ELFError: Magic number does not match

Copy link
Copy Markdown
Member
@peace-maker peace-maker left a comment

Choose a reason for hiding this comment

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

Nice, thank you!

@peace-maker peace-maker merged commit d7817a7 into Gallopsled:dev Sep 11, 2024
@tesuji tesuji deleted the checksec-not-elf branch September 11, 2024 15:55
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

0