Skip to content

Commit 0880b57

Browse files
authored
Merge pull request #1 from hvitved/csharp/no-ternary-cond-taint-tracking
C#: No taint-tracking steps for ternary conditionals
2 parents ca08097 + 9dde1ce commit 0880b57

File tree

4 files changed

+5
-230
lines changed

4 files changed

+5
-230
lines changed

csharp/ql/src/semmle/code/csharp/dataflow/internal/TaintTrackingPrivate.qll

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,11 @@ private class LocalTaintExprStepConfiguration extends ControlFlowReachabilityCon
9090
isSuccessor = true
9191
)
9292
or
93-
e1 = e2.(LogicalOperation).getAnOperand() and
93+
e1 = e2.(UnaryLogicalOperation).getAnOperand() and
94+
scope = e2 and
95+
isSuccessor = false
96+
or
97+
e1 = e2.(BinaryLogicalOperation).getAnOperand() and
9498
scope = e2 and
9599
isSuccessor = false
96100
or

0 commit comments

Comments
 (0)