hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2026-05-02 20:25:13 +02:00
Code Issues Packages Projects Releases Wiki Activity

use a consistent comment about the ignore case flag

Browse Source
This commit is contained in:
Erik Krogh Kristensen
2021-09-01 12:46:50 +02:00
parent ff74fe1e03
commit 537450606e
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
javascript/ql/lib/semmle/javascript/security/performance/ReDoSUtil.qll
View File

@@ -201,7 +201,7 @@ private newtype TInputSymbol =
not RegExpFlags::isIgnoreCase(cc.getRootTerm())
).getValue().charAt(_)
or
// normalize to lower case if the regexp is case insensitive
// normalize everything to lower case if the regexp is case insensitive
c =
any(RegexpCharacterConstant cc, string char |
cc instanceof RelevantRegExpTerm and
@@ -577,7 +577,7 @@ predicate delta(State q1, EdgeLabel lbl, State q2) {
not RegExpFlags::isIgnoreCase(s.getRootTerm()) and
lbl = Char(s.getValue().charAt(i))
or
// normalizing to lower case if ignorecase flag is set
// normalize everything to lower case if the regexp is case insensitive
RegExpFlags::isIgnoreCase(s.getRootTerm()) and
exists(string c | c = s.getValue().charAt(i) | lbl = Char(c.toLowerCase()))
) and