Skip to content

Commit 2523946

Browse files
committed
sync files
1 parent e7aef17 commit 2523946

File tree

3 files changed

+12
-3
lines changed

3 files changed

+12
-3
lines changed

java/ql/lib/semmle/code/java/security/OverlyLargeRangeQuery.qll

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,10 @@ class OverlyWideRange extends RegExpCharacterRange {
9696
toCodePoint("A") <= high
9797
or
9898
// a non-alphanumeric char as part of the range boundaries
99-
exists(int bound | bound = [low, high] | not isAlphanumeric(bound.toUnicode()))
99+
exists(int bound | bound = [low, high] | not isAlphanumeric(bound.toUnicode())) and
100+
// while still being ascii
101+
low < 128 and
102+
high < 128
100103
) and
101104
// allowlist for known ranges
102105
not this = allowedWideRanges()

python/ql/lib/semmle/python/security/OverlyLargeRangeQuery.qll

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,10 @@ class OverlyWideRange extends RegExpCharacterRange {
9696
toCodePoint("A") <= high
9797
or
9898
// a non-alphanumeric char as part of the range boundaries
99-
exists(int bound | bound = [low, high] | not isAlphanumeric(bound.toUnicode()))
99+
exists(int bound | bound = [low, high] | not isAlphanumeric(bound.toUnicode())) and
100+
// while still being ascii
101+
low < 128 and
102+
high < 128
100103
) and
101104
// allowlist for known ranges
102105
not this = allowedWideRanges()

ruby/ql/lib/codeql/ruby/security/OverlyLargeRangeQuery.qll

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,10 @@ class OverlyWideRange extends RegExpCharacterRange {
9696
toCodePoint("A") <= high
9797
or
9898
// a non-alphanumeric char as part of the range boundaries
99-
exists(int bound | bound = [low, high] | not isAlphanumeric(bound.toUnicode()))
99+
exists(int bound | bound = [low, high] | not isAlphanumeric(bound.toUnicode())) and
100+
// while still being ascii
101+
low < 128 and
102+
high < 128
100103
) and
101104
// allowlist for known ranges
102105
not this = allowedWideRanges()

0 commit comments

Comments
 (0)