Skip to content

Conversation

chocolatkey
Copy link
Member

@chocolatkey chocolatkey commented Oct 3, 2024

Some broken publications have an invalid font obfuscation key, and this causes a panic due to a divide by zero in the deobfuscateFont function. This adds a check for that

@chocolatkey chocolatkey changed the title add safety check for 0-length obfuscationKey Add safety check for invalid font obfuscation key Oct 3, 2024
Copy link
Member

@mickael-menu mickael-menu left a comment

Choose a reason for hiding this comment

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

LGTM 🙏

@chocolatkey chocolatkey merged commit 6c772de into feature-read-compressed Oct 9, 2024
2 checks passed
@chocolatkey chocolatkey deleted the safety-check-font-deobfuscation branch October 9, 2024 11:28
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