diff options
author | Daniel Stenberg <daniel@haxx.se> | 2018-05-18 16:48:13 +0200 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2018-05-18 23:33:44 +0200 |
commit | 404c8850da5a677638959f4e38bb7692cb887d3a (patch) | |
tree | 0cc2d3399ba189df38e83536b42a1f837da27aec /tests/unit | |
parent | 27aebcc1d1a0825af6e812f86bf5eaf510f610d1 (diff) |
curl_fnmatch: only allow two asterisks for matching
The previous limit of 5 can still end up in situation that takes a very
long time and consumes a lot of CPU.
If there is still a rare use case for this, a user can provide their own
fnmatch callback for a version that allows a larger set of wildcards.
This commit was triggered by yet another OSS-Fuzz timeout due to this.
Bug: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=8369
Closes #2587
Diffstat (limited to 'tests/unit')
-rw-r--r-- | tests/unit/unit1307.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/tests/unit/unit1307.c b/tests/unit/unit1307.c index 0d2257bf0..5f60332b8 100644 --- a/tests/unit/unit1307.c +++ b/tests/unit/unit1307.c @@ -185,11 +185,7 @@ static const struct testcase tests[] = { { "\\?.txt", "x.txt", NOMATCH }, { "\\*.txt", "x.txt", NOMATCH }, { "\\*\\\\.txt", "*\\.txt", MATCH }, - { "*\\**\\?*\\\\*", "cc*cc?cc\\cc*cc", MATCH }, { "*\\**\\?*\\\\*", "cc*cc?cccc", NOMATCH }, - { "*\\**\\?*\\\\*", "cc*cc?cc\\cc*cc", MATCH }, - { "*\\?*\\**", "cc?c*c", MATCH }, - { "*\\?*\\**curl*", "cc?c*curl", MATCH }, { "*\\?*\\**", "cc?cc", NOMATCH }, { "\\\"\\$\\&\\'\\(\\)", "\"$&'()", MATCH }, { "\\*\\?\\[\\\\\\`\\|", "*?[\\`|", MATCH }, |