Skip to content

Fix circular composite glyph path resolution#823

Closed
fdb wants to merge 1 commit intomasterfrom
codex/fix-composite-glyph-cycle
Closed

Fix circular composite glyph path resolution#823
fdb wants to merge 1 commit intomasterfrom
codex/fix-composite-glyph-cycle

Conversation

@fdb
Copy link
Copy Markdown
Contributor

@fdb fdb commented Mar 26, 2026

Summary

  • detect re-entrant TrueType glyph path loading and throw a controlled error for circular composite references
  • add regression coverage for cyclic composite glyphs in normal and lowMemory parsing modes
  • include a minimal malicious TrueType fixture and generator script to reproduce the issue deterministically

Testing

  • npm test

@fdb
Copy link
Copy Markdown
Contributor Author

fdb commented Apr 3, 2026

Closing as this is superseded by #826 which was merged with a similar fix (cycle detection in glyf.mjs using a per-GlyphSet WeakMap). Thanks for the original work on this — it informed the final approach.

@fdb fdb closed this Apr 3, 2026
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.

1 participant