Skip to content

Commit 3766678

Browse files
committed
move RegexpMetaChars into Regexp.qll
1 parent 33641c8 commit 3766678

File tree

2 files changed

+12
-9
lines changed

2 files changed

+12
-9
lines changed

javascript/ql/src/semmle/javascript/Regexp.qll

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1237,8 +1237,18 @@ module RegExp {
12371237
}
12381238
}
12391239

1240-
private class DefaultMetaCharacter extends MetaCharacter {
1241-
DefaultMetaCharacter() { this = ["<", "'", "\""] }
1240+
/**
1241+
* A meta character used by HTML.
1242+
*/
1243+
private class HTMLMetaCharacter extends MetaCharacter {
1244+
HTMLMetaCharacter() { this = ["<", "'", "\""] }
1245+
}
1246+
1247+
/**
1248+
* A meta character used by regular expressions.
1249+
*/
1250+
private class RegexpMetaChars extends RegExp::MetaCharacter {
1251+
RegexpMetaChars() { this = ["{", "[", "+"] }
12421252
}
12431253

12441254
/**

javascript/ql/src/semmle/javascript/security/dataflow/RegExpInjectionCustomizations.qll

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -71,11 +71,4 @@ module RegExpInjection {
7171
)
7272
}
7373
}
74-
75-
/**
76-
* Meta characters used in the above sanitizer.
77-
*/
78-
private class RegexpMetaChars extends RegExp::MetaCharacter {
79-
RegexpMetaChars() { this = ["{", "[", "+"] }
80-
}
8174
}

0 commit comments

Comments
 (0)