diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2018-05-18 16:48:13 +0200 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2018-05-18 16:48:13 +0200 |
| commit | cea7a1e846c169d6ae3be9617c3a68c41f8c527e (patch) | |
| tree | ed989cf72895c77aebff16a430564779bcafb160 /tests/unit/unit1307.c | |
| parent | ccf85d4c2b1b2fac2f6cbac31d719bfa73b76ed2 (diff) | |
| download | curl-bagder/fnmatch-limit-wildcard.tar.gz | |
curl_fnmatch: only allow two asterisks for matchingbagder/fnmatch-limit-wildcard
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.
Diffstat (limited to 'tests/unit/unit1307.c')
| -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 }, |