chore(deps): update dependency fast-xml-parser to v4.5.4 [security]#286
Open
renovate[bot] wants to merge 1 commit intothe-onefrom
Open
chore(deps): update dependency fast-xml-parser to v4.5.4 [security]#286renovate[bot] wants to merge 1 commit intothe-onefrom
renovate[bot] wants to merge 1 commit intothe-onefrom
Conversation
Contributor
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the
Comment |
9e0a960 to
635e4ef
Compare
635e4ef to
ae62fd1
Compare
ae62fd1 to
4f76cb9
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
4.5.1→4.5.4GitHub Vulnerability Alerts
CVE-2026-25896
Entity encoding bypass via regex injection in DOCTYPE entity names
Summary
A dot (
.) in a DOCTYPE entity name is treated as a regex wildcard during entity replacement, allowing an attacker to shadow built-in XML entities (<,>,&,",') with arbitrary values. This bypasses entity encoding and leads to XSS when parsed output is rendered.Details
The fix for CVE-2023-34104 addressed some regex metacharacters in entity names but missed
.(period), which is valid in XML names per the W3C spec.In
DocTypeReader.js, entity names are passed directly toRegExp():An entity named
l.produces the regex/&l.;/gwhere.matches any character, including thetin<. Since DOCTYPE entities are replaced before built-in entities, this shadows<entirely.The same issue exists in
OrderedObjParser.js:81(addExternalEntities), and in the v6 codebase -EntitiesParser.jshas avalidateEntityNamefunction with a character blacklist, but.is not included:Shadowing all 5 built-in entities
l./&l.;/g<g./&g.;/g>am./&am.;/g&quo./&quo.;/g"apo./&apo.;/g'PoC
No special parser options needed -
processEntities: trueis the default.When an app renders
result.root.textin a page (e.g.innerHTML, template interpolation, SSR), the injected<img onerror>fires.&can be shadowed too:Impact
This is a complete bypass of XML entity encoding. Any application that parses untrusted XML and uses the output in HTML, SQL, or other injection-sensitive contexts is affected.
</>/&/"/'with arbitrary stringsSuggested fix
Escape regex metacharacters before constructing the replacement regex:
For v6, add
.to the blacklist invalidateEntityName:Severity
CWE-185 (Incorrect Regular Expression)
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:C/C:L/I:H/A:N - 9.3 (CRITICAL)
Entity decoding is a fundamental trust boundary in XML processing. This completely undermines it with no preconditions.
CVE-2026-26278
Summary
The XML parser can be forced to do an unlimited amount of entity expansion. With a very small XML input, it’s possible to make the parser spend seconds or even minutes processing a single request, effectively freezing the application.
Details
There is a check in
DocTypeReader.jsthat tries to prevent entity expansion attacks by rejecting entities that reference other entities (it looks for & inside entity values). This does stop classic “Billion Laughs” payloads.However, it doesn’t stop a much simpler variant.
If you define one large entity that contains only raw text (no & characters) and then reference it many times, the parser will happily expand it every time. There is no limit on how large the expanded result can become, or how many replacements are allowed.
The problem is in
replaceEntitiesValue()insideOrderedObjParser.js. It repeatedly runsval.replace()in a loop, without any checks on total output size or execution cost. As the entity grows or the number of references increases, parsing time explodes.Relevant code:
DocTypeReader.js(lines 28–33): entity registration only checks for &OrderedObjParser.js(lines 439–458): entity replacement loop with no limitsPoC
Impact
This is a straightforward denial-of-service issue.
Any service that parses user-supplied XML using the default configuration is vulnerable. Since Node.js runs on a single thread, the moment the parser starts expanding entities, the event loop is blocked. While this is happening, the server can’t handle any other requests.
In testing, a payload of only a few kilobytes was enough to make a simple HTTP server completely unresponsive for several minutes, with all other requests timing out.
Workaround
Avoid using DOCTYPE parsing by
processEntities: falseoption.CVE-2026-27942
Impact
Application crashes with stack overflow when user use XML builder with
prserveOrder:truefor following or similar inputCause:
arrToStrwas not validating if the input is an array or a string and treating all non-array values as text content.What kind of vulnerability is it? Who is impacted?
Patches
Yes in 5.3.8
Workarounds
Use XML builder with
preserveOrder:falseor check the input data before passing to builder.References
Are there any links users can visit to find out more?
Release Notes
NaturalIntelligence/fast-xml-parser (fast-xml-parser)
v4.5.4Compare Source
v4.5.3Compare Source
v4.5.2Compare Source
Configuration
📅 Schedule: Branch creation - "" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.