Skip to content

Raise totalEntitySizeLimit for dictionary XML parsing#3

Merged
ghedwards merged 1 commit into
masterfrom
fix/dictionary-entity-size-limit
Jul 6, 2026
Merged

Raise totalEntitySizeLimit for dictionary XML parsing#3
ghedwards merged 1 commit into
masterfrom
fix/dictionary-entity-size-limit

Conversation

@ghedwards

@ghedwards ghedwards commented Jul 6, 2026

Copy link
Copy Markdown

Summary

Test plan

  • Reproduced the exact JAXP00010004 failure in isolation with a small synthetic DTD (repeated small entity, same shape as html.xml's pattern) on JDK 25, confirmed it disappears with the property set and the parser instance without it still fails as before (proving the fix is scoped, not a JVM-wide change).
  • Maintainer to confirm dictionary loading still succeeds end-to-end via this repo's own test/build (this repo's Gradle 6.9.4 wrapper isn't compatible with the JDKs available where this PR was authored).

html.xml's shared attribute-group entities (event handlers, core/lang
attributes) are &entity;-referenced 335 times across ~90 tag definitions,
which legitimately accumulates past newer JDKs' tightened default for
jdk.xml.totalEntitySizeLimit (confirmed failing on JDK 25, fine on JDK 21).
Set it explicitly on this parser instance only, so other XML parsing in a
consuming application is unaffected.

Fixes #2
@ghedwards ghedwards merged commit d5f760e into master Jul 6, 2026
3 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.

SAXParseException (JAXP00010004) loading html.xml dictionary on JDK 25 — accumulated entity expansion exceeds jdk.xml.totalEntitySizeLimit

2 participants