chore(deps): update dependency eslint-plugin-import@2.29.1>minimatch to v3.1.4 [security]#56
Open
renovate[bot] wants to merge 1 commit into
Conversation
adf4933 to
238a381
Compare
238a381 to
81d9924
Compare
81d9924 to
6a18629
Compare
…to v3.1.4 [security]
6a18629 to
dbd2514
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:
3.1.3→3.1.4minimatch ReDoS: nested *() extglobs generate catastrophically backtracking regular expressions
CVE-2026-27904 / GHSA-23c5-xmqv-rm74
More information
Details
Summary
Nested
*()extglobs produce regexps with nested unbounded quantifiers (e.g.(?:(?:a|b)*)*), which exhibit catastrophic backtracking in V8. With a 12-byte pattern*(*(*(a|b)))and an 18-byte non-matching input,minimatch()stalls for over 7 seconds. Adding a single nesting level or a few input characters pushes this to minutes. This is the most severe finding: it is triggered by the defaultminimatch()API with no special options, and the minimum viable pattern is only 12 bytes. The same issue affects+()extglobs equally.Details
The root cause is in
AST.toRegExpSource()atsrc/ast.ts#L598. For the*extglob type, the close token emitted is)*or)?, wrapping the recursive body in(?:...)*. When extglobs are nested, each level adds another*quantifier around the previous group:This produces the following regexps:
*(a|b)/^(?:a|b)*$/*(*(a|b))/^(?:(?:a|b)*)*$/*(*(*(a|b)))/^(?:(?:(?:a|b)*)*)*$/*(*(*(*(a|b))))/^(?:(?:(?:(?:a|b)*)*)*)*$/These are textbook nested-quantifier patterns. Against an input of repeated
acharacters followed by a non-matching characterz, V8's backtracking engine explores an exponential number of paths before returningfalse.The generated regex is stored on
this.setand evaluated insidematchOne()atsrc/index.ts#L1010viap.test(f). It is reached through the standardminimatch()call with no configuration.Measured times via
minimatch():*(*(a|b))ax30 +z*(*(*(a|b)))ax20 +z*(*(*(*(a|b))))ax25 +z*(a|a)ax25 +zDepth inflection at fixed input
ax16 +z:*(a|b)*(*(a|b))*(*(*(a|b)))*(*(*(*(a|b))))Going from depth 2 to depth 3 with a 20-character input jumps from 66ms to 123,544ms -- a 1,867x increase from a single added nesting level.
PoC
Tested on minimatch@10.2.2, Node.js 20.
Step 1 -- verify the generated regexps and timing (standalone script)
Save as
poc4-validate.mjsand run withnode poc4-validate.mjs:Observed output:
Step 2 -- HTTP server (event loop starvation proof)
Save as
poc4-server.mjs:Terminal 1 -- start the server:
Terminal 2 -- fire the attack (depth=3, 19 a's + z) and return immediately:
Terminal 3 -- send a benign request while the attack is in-flight:
Observed output -- Terminal 2 (attack):
Observed output -- Terminal 3 (benign, concurrent):
Terminal 1 (server log):
The server reports
"ms":"0"for the benign request -- the legitimate request itself requires no CPU time. The entire 63-secondtime_totalis time spent waiting for the event loop to be released. The benign request was only dispatched after the attack completed, confirmed by the server log timestamps.Note: standalone script timing (~7s at n=19) is lower than server timing (64s) because the standalone script had warmed up V8's JIT through earlier sequential calls. A cold server hits the worst case. Both measurements confirm catastrophic backtracking -- the server result is the more realistic figure for production impact.
Impact
Any context where an attacker can influence the glob pattern passed to
minimatch()is vulnerable. The realistic attack surface includes build tools and task runners that accept user-supplied glob arguments, multi-tenant platforms where users configure glob-based rules (file filters, ignore lists, include patterns), and CI/CD pipelines that evaluate user-submitted config files containing glob expressions. No evidence was found of production HTTP servers passing raw user input directly as the extglob pattern, so that framing is not claimed here.Depth 3 (
*(*(*(a|b))), 12 bytes) stalls the Node.js event loop for 7+ seconds with an 18-character input. Depth 2 (*(*(a|b)), 9 bytes) reaches 68 seconds with a 31-character input. Both the pattern and the input fit in a query string or JSON body without triggering the 64 KB length guard.+()extglobs share the same code path and produce equivalent worst-case behavior (6.3 seconds at depth=3 with an 18-character input, confirmed).Mitigation available: passing
{ noext: true }tominimatch()disables extglob processing entirely and reduces the same input to 0ms. Applications that do not need extglob syntax should set this option when handling untrusted patterns.Severity
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:HReferences
This data is provided by the GitHub Advisory Database (CC-BY 4.0).
Release Notes
isaacs/minimatch (eslint-plugin-import@2.29.1>minimatch)
v3.1.4Compare Source
Configuration
📅 Schedule: (UTC)
🚦 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.