[SECURITY] Update dependency fast-xml-parser to v5.5.7#296
Open
specmatic-builder wants to merge 1 commit intomasterfrom
Open
[SECURITY] Update dependency fast-xml-parser to v5.5.7#296specmatic-builder wants to merge 1 commit intomasterfrom
specmatic-builder wants to merge 1 commit intomasterfrom
Conversation
057a5d9 to
71220bd
Compare
128e164 to
27b9fa0
Compare
27b9fa0 to
73997ed
Compare
73997ed to
2c14d90
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:
5.3.6→5.5.7GitHub Vulnerability Alerts
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?
CVE-2026-33036
Summary
The fix for CVE-2026-26278 added entity expansion limits (
maxTotalExpansions,maxExpandedLength,maxEntityCount,maxEntitySize) to prevent XML entity expansion Denial of Service. However, these limits are only enforced for DOCTYPE-defined entities. Numeric character references (&#NNN;and&#xHH;) and standard XML entities (<,>, etc.) are processed through a separate code path that does NOT enforce any expansion limits.An attacker can use massive numbers of numeric entity references to completely bypass all configured limits, causing excessive memory allocation and CPU consumption.
Affected Versions
fast-xml-parser v5.x through v5.5.3 (and likely v5.5.5 on npm)
Root Cause
In
src/xmlparser/OrderedObjParser.js, thereplaceEntitiesValue()function has two separate entity replacement loops:entityExpansionCountandcurrentExpandedLengthtracking. This was the CVE-2026-26278 fix.lastEntitiesloop — replaces standard entities includingnum_dec(/&#([0-9]{1,7});/g) andnum_hex(/&#x([0-9a-fA-F]{1,6});/g). This loop has NO expansion counting at all.The numeric entity regex replacements at lines 97-98 are part of
lastEntitiesand go through the uncounted loop, completely bypassing the CVE-2026-26278 fix.Proof of Concept
Results:
&#​65;references → 500,000 char output (5x default maxExpandedLength of 100,000)maxTotalExpansions=10andmaxExpandedLength=100, 10K references produce 50,000 charsA) exhibit the same bypassImpact
Denial of Service — An attacker who can provide XML input to applications using fast-xml-parser can cause:
This is particularly dangerous because the application developer may have explicitly configured strict entity expansion limits believing they are protected, while numeric entities silently bypass all of them.
Suggested Fix
Apply the same
entityExpansionCountandcurrentExpandedLengthtracking to thelastEntitiesloop (lines 674-677) and the HTML entities loop (lines 680-686), similar to how DOCTYPE entities are tracked at lines 638-670.Workaround
Set
htmlEntities:falseCVE-2026-33349
Summary
The
DocTypeReaderin fast-xml-parser uses JavaScript truthy checks to evaluatemaxEntityCountandmaxEntitySizeconfiguration limits. When a developer explicitly sets either limit to0— intending to disallow all entities or restrict entity size to zero bytes — the falsy nature of0in JavaScript causes the guard conditions to short-circuit, completely bypassing the limits. An attacker who can supply XML input to such an application can trigger unbounded entity expansion, leading to memory exhaustion and denial of service.Details
The
OptionsBuilder.jscorrectly preserves a user-supplied value of0using nullish coalescing (??):However,
DocTypeReader.jsuses truthy evaluation to check these limits. Because0is falsy in JavaScript, the entire guard expression short-circuits tofalse, and the limit is never enforced:The execution flow is:
processEntities: { maxEntityCount: 0, maxEntitySize: 0 }intending to block all entity definitions.OptionsBuilder.normalizeProcessEntitiespreserves the0values via??(correct behavior).DocTypeReader.readDocTypeevaluatesthis.options.maxEntityCount && ...— since0is falsy, the entire condition isfalse.DocTypeReader.readEntityExpevaluatesthis.options.maxEntitySize && ...— same result.PoC
Expected output:
Impact
maxEntityCount: 0ormaxEntitySize: 0, intending to prohibit entities entirely.0receive no protection — the opposite of their intent. This creates a false sense of security.0are affected. The default configuration (maxEntityCount: 100,maxEntitySize: 10000) is not vulnerable. Theenabled: falseoption correctly disables entity processing entirely and is not affected.Recommended Fix
Replace the truthy checks in
DocTypeReader.jswith explicit type checks that correctly treat0as a valid numeric limit:Workaround
If you don't want to processed the entities, keep the processEntities flag to false instead of setting any limit to 0.
fast-xml-parser has stack overflow in XMLBuilder with preserveOrder
CVE-2026-27942 / GHSA-fj3w-jwp8-x2g3
More information
Details
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?
Severity
CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:N/VI:N/VA:L/SC:N/SI:N/SA:N/E:UReferences
This data is provided by OSV and the GitHub Advisory Database (CC-BY 4.0).
fast-xml-parser affected by numeric entity expansion bypassing all entity expansion limits (incomplete fix for CVE-2026-26278)
CVE-2026-33036 / GHSA-8gc5-j5rx-235r
More information
Details
Summary
The fix for CVE-2026-26278 added entity expansion limits (
maxTotalExpansions,maxExpandedLength,maxEntityCount,maxEntitySize) to prevent XML entity expansion Denial of Service. However, these limits are only enforced for DOCTYPE-defined entities. Numeric character references (&#NNN;and&#xHH;) and standard XML entities (<,>, etc.) are processed through a separate code path that does NOT enforce any expansion limits.An attacker can use massive numbers of numeric entity references to completely bypass all configured limits, causing excessive memory allocation and CPU consumption.
Affected Versions
fast-xml-parser v5.x through v5.5.3 (and likely v5.5.5 on npm)
Root Cause
In
src/xmlparser/OrderedObjParser.js, thereplaceEntitiesValue()function has two separate entity replacement loops:entityExpansionCountandcurrentExpandedLengthtracking. This was the CVE-2026-26278 fix.lastEntitiesloop — replaces standard entities includingnum_dec(/&#([0-9]{1,7});/g) andnum_hex(/&#x([0-9a-fA-F]{1,6});/g). This loop has NO expansion counting at all.The numeric entity regex replacements at lines 97-98 are part of
lastEntitiesand go through the uncounted loop, completely bypassing the CVE-2026-26278 fix.Proof of Concept
Results:
&#​65;references → 500,000 char output (5x default maxExpandedLength of 100,000)maxTotalExpansions=10andmaxExpandedLength=100, 10K references produce 50,000 charsA) exhibit the same bypassImpact
Denial of Service — An attacker who can provide XML input to applications using fast-xml-parser can cause:
This is particularly dangerous because the application developer may have explicitly configured strict entity expansion limits believing they are protected, while numeric entities silently bypass all of them.
Suggested Fix
Apply the same
entityExpansionCountandcurrentExpandedLengthtracking to thelastEntitiesloop (lines 674-677) and the HTML entities loop (lines 680-686), similar to how DOCTYPE entities are tracked at lines 638-670.Workaround
Set
htmlEntities:falseSeverity
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:HReferences
This data is provided by OSV and the GitHub Advisory Database (CC-BY 4.0).
Entity Expansion Limits Bypassed When Set to Zero Due to JavaScript Falsy Evaluation in fast-xml-parser
CVE-2026-33349 / GHSA-jp2q-39xq-3w4g
More information
Details
Summary
The
DocTypeReaderin fast-xml-parser uses JavaScript truthy checks to evaluatemaxEntityCountandmaxEntitySizeconfiguration limits. When a developer explicitly sets either limit to0— intending to disallow all entities or restrict entity size to zero bytes — the falsy nature of0in JavaScript causes the guard conditions to short-circuit, completely bypassing the limits. An attacker who can supply XML input to such an application can trigger unbounded entity expansion, leading to memory exhaustion and denial of service.Details
The
OptionsBuilder.jscorrectly preserves a user-supplied value of0using nullish coalescing (??):However,
DocTypeReader.jsuses truthy evaluation to check these limits. Because0is falsy in JavaScript, the entire guard expression short-circuits tofalse, and the limit is never enforced:The execution flow is:
processEntities: { maxEntityCount: 0, maxEntitySize: 0 }intending to block all entity definitions.OptionsBuilder.normalizeProcessEntitiespreserves the0values via??(correct behavior).DocTypeReader.readDocTypeevaluatesthis.options.maxEntityCount && ...— since0is falsy, the entire condition isfalse.DocTypeReader.readEntityExpevaluatesthis.options.maxEntitySize && ...— same result.PoC
Expected output:
Impact
maxEntityCount: 0ormaxEntitySize: 0, intending to prohibit entities entirely.0receive no protection — the opposite of their intent. This creates a false sense of security.0are affected. The default configuration (maxEntityCount: 100,maxEntitySize: 10000) is not vulnerable. Theenabled: falseoption correctly disables entity processing entirely and is not affected.Recommended Fix
Replace the truthy checks in
DocTypeReader.jswith explicit type checks that correctly treat0as a valid numeric limit:Workaround
If you don't want to processed the entities, keep the processEntities flag to false instead of setting any limit to 0.
Severity
CVSS:3.1/AV:N/AC:H/PR:N/UI:N/S:U/C:N/I:N/A:HReferences
This data is provided by OSV and the GitHub Advisory Database (CC-BY 4.0).
Release Notes
NaturalIntelligence/fast-xml-parser (fast-xml-parser)
v5.5.7Compare Source
v5.5.6: fix entity expansion and incorrect replacement and performanceCompare Source
Full Changelog: NaturalIntelligence/fast-xml-parser@v5.5.5...v5.5.6
v5.5.5: support onDangerousPropertyCompare Source
Full Changelog: NaturalIntelligence/fast-xml-parser@v5.5.3...v5.5.5
v5.5.4Compare Source
v5.5.3Compare Source
v5.5.2: update dependecies to fix typingsCompare Source
Full Changelog: NaturalIntelligence/fast-xml-parser@v5.5.1...v5.5.2
v5.5.1: integrate path-expression-matcherCompare Source
v5.5.0Compare Source
v5.4.2Compare Source
v5.4.1Compare Source
v5.4.0: Separate BuilderCompare Source
XML Builder was the part of fast-xml-parser for years. But considering that any bug in builder may false-alarm the users who are only using parser and vice-versa, we have decided to split it into a separate package.
Migration
To migrate to fast-xml-builder;
From
To
XMLBuilder will be removed from current package in any next major version of this library. So better to migrate.
v5.3.9: support strictReservedNamesCompare Source
Full Changelog: NaturalIntelligence/fast-xml-parser@v5.3.9...v5.3.9
v5.3.8: handle non-array input for XML builder && support maxNestedTagsCompare Source
Full Changelog: NaturalIntelligence/fast-xml-parser@v5.3.7...v5.3.8
v5.3.7: CJS typing fixCompare Source
What's Changed
X2jOptionsat declaration site by @Drarig29 in #787New Contributors
Full Changelog: NaturalIntelligence/fast-xml-parser@v5.3.6...v5.3.7
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 has been generated by Renovate Bot. Specmatic specific configuration is managed here.