Rule 942360: Small bug in the regex cause FP
See original GitHub issue_Issue originally created by user Taiki-San on date 2019-10-03 15:24:51. Link to original issue: https://github.com/SpiderLabs/owasp-modsecurity-crs/issues/1580._
Type of Issue
False positive.
Description
The rule trigger a false positive for the following pattern: \W asbla from
.
More specifically, patterns similar to , aside from
will trigger.
This is likely due to the following construct [\W]\s+as\s*?[\\"'`\w]+\s*?from
being used instead [\W]\s+as\s+?[\\"'`\w]+\s*?from
.
Specifically \s*?
instead of \s+?
.
Confirmation
[X] I have removed any personal data (email addresses, IP addresses, passwords, domain names) from any logs posted.
Issue Analytics
- State:
- Created 3 years ago
- Comments:10
Top Results From Across the Web
Rule 942360: Small bug in the regex cause FP #1580 - GitHub
The rule trigger a false positive for the following pattern: \W asbla from . More specifically, patterns similar to , aside from will...
Read more >Disassembling SQLi Rules
This module assembles multiple regular expressions into one regular expression. The source patterns were lost over the years as they were taken ...
Read more >changes - GitHub
OWASP ModSecurity Core Rule Set (CRS) CHANGES == == Report Bugs/Issues to ... Add regression tests (Franziska Bühler) * Fix FP with create...
Read more >Demystifying regular expression bugs - Chris Brown
Regular expressions cause string-related bugs and open security ... However, beyond ReDoS (Regular expression Denial of Service), little is known.
Read more >[2104.09693] Demystifying Regular Expression Bugs - arXiv
Demystifying Regular Expression Bugs: A comprehensive study on regular expression bug causes, fixes, and testing. Authors:Peipei Wang, Chris ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
User fgsch commented on date 2019-10-28 15:52:25:
Sorry for the delay. The fix is merged now.
User emphazer commented on date 2019-10-08 10:54:49:
fgsch good solution with \b instead of +. okay, i could change the regression test to
6 As" from
for example