Skip to content

Commit 52abf3b

Browse files
authored
Merge pull request #19997 from hvitved/java/use-mad-in-log-injection-test
Java: Use MaD in log injection test
2 parents e5b4a15 + 6fdec47 commit 52abf3b

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
extensions:
2+
- addsTo:
3+
pack: codeql/java-all
4+
extensible: sourceModel
5+
data:
6+
- ["loginjection", "LogInjectionTest", False, "source", "()", "", "ReturnValue", "remote", "manual"]

java/ql/test/query-tests/security/CWE-117/LogInjectionTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
package loginjection;
2+
13
import java.util.ResourceBundle;
24
import java.util.logging.LogRecord;
35
import java.util.regex.Pattern;
Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,4 @@
11
import java
22
import semmle.code.java.security.LogInjectionQuery
33
import utils.test.InlineFlowTest
4-
5-
private class TestSource extends RemoteFlowSource {
6-
TestSource() { this.asExpr().(MethodCall).getMethod().hasName("source") }
7-
8-
override string getSourceType() { result = "test source" }
9-
}
10-
114
import TaintFlowTest<LogInjectionConfig>

0 commit comments

Comments
 (0)