Skip to content

Commit 195ed01

Browse files
authored
Merge pull request github#5393 from aschackmull/java/taint-not-value-step
Java: Remove value steps from taint steps.
2 parents 0274162 + 1d3ad0c commit 195ed01

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

java/ql/src/semmle/code/java/dataflow/internal/TaintTrackingUtil.qll

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,8 @@ predicate localAdditionalTaintStep(DataFlow::Node src, DataFlow::Node sink) {
4646
localAdditionalTaintUpdateStep(src.asExpr(),
4747
sink.(DataFlow::PostUpdateNode).getPreUpdateNode().asExpr())
4848
or
49-
summaryStep(src, sink, "taint")
49+
summaryStep(src, sink, "taint") and
50+
not summaryStep(src, sink, "value")
5051
or
5152
exists(Argument arg |
5253
src.asExpr() = arg and

0 commit comments

Comments
 (0)