Skip to content

Commit dc83b62

Browse files
committed
Allow MaD sanitizers for java/log-injection and java/sensitive-log
1 parent 2e685e7 commit dc83b62

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,10 @@ private class DefaultLogInjectionSink extends LogInjectionSink {
3636
private class DefaultLogInjectionSanitizer extends LogInjectionSanitizer instanceof SimpleTypeSanitizer
3737
{ }
3838

39+
private class ExternalLogInjectionSanitizer extends LogInjectionSanitizer {
40+
ExternalLogInjectionSanitizer() { barrierNode(this, "log-injection") }
41+
}
42+
3943
private class LineBreaksLogInjectionSanitizer extends LogInjectionSanitizer {
4044
LineBreaksLogInjectionSanitizer() {
4145
logInjectionSanitizer(this.asExpr())

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,10 @@ private class DefaultSensitiveLoggerBarrier extends SensitiveLoggerBarrier {
120120
}
121121
}
122122

123+
private class ExternalSensitiveLoggerBarrier extends SensitiveLoggerBarrier {
124+
ExternalSensitiveLoggerBarrier() { barrierNode(this, "log-injection") }
125+
}
126+
123127
/** A data-flow configuration for identifying potentially-sensitive data flowing to a log output. */
124128
module SensitiveLoggerConfig implements DataFlow::ConfigSig {
125129
predicate isSource(DataFlow::Node source) { source instanceof SensitiveLoggerSource }

0 commit comments

Comments
 (0)