Skip to content

Commit 81ad10b

Browse files
Jami CogswellJami Cogswell
authored andcommitted
update sink names
1 parent 5b089bb commit 81ad10b

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,13 @@ private import semmle.code.java.frameworks.Regex
66
private import semmle.code.java.frameworks.apache.Lang
77

88
/** A data flow sink for untrusted user input used to construct regular expressions. */
9-
abstract class Sink extends DataFlow::ExprNode { }
9+
abstract class RegexInjectionSink extends DataFlow::ExprNode { }
1010

1111
/** A sanitizer for untrusted user input used to construct regular expressions. */
1212
abstract class RegexInjectionSanitizer extends DataFlow::ExprNode { }
1313

14-
private class RegexInjectionSink extends Sink {
15-
RegexInjectionSink() {
14+
private class DefaultRegexInjectionSink extends RegexInjectionSink {
15+
DefaultRegexInjectionSink() {
1616
exists(MethodAccess ma, Method m | m = ma.getMethod() |
1717
ma.getArgument(0) = this.asExpr() and
1818
(

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ class RegexInjectionConfiguration extends TaintTracking::Configuration {
1111

1212
override predicate isSource(DataFlow::Node source) { source instanceof RemoteFlowSource }
1313

14-
override predicate isSink(DataFlow::Node sink) { sink instanceof Sink }
14+
override predicate isSink(DataFlow::Node sink) { sink instanceof RegexInjectionSink }
1515

1616
override predicate isSanitizer(DataFlow::Node node) { node instanceof RegexInjectionSanitizer }
1717
}

0 commit comments

Comments
 (0)