Merge pull request #2497 from hvitved/csharp/remove-cp

jbj · web-flow · commit 0012fef5041f · 2019-12-06T13:58:33.000Z
C#: Remove a Cartesian product
diff --git a/csharp/ql/src/semmle/code/csharp/dataflow/internal/TaintTrackingPrivate.qll b/csharp/ql/src/semmle/code/csharp/dataflow/internal/TaintTrackingPrivate.qll
@@ -117,6 +117,7 @@ private class LocalTaintExprStepConfiguration extends ControlFlowReachabilityCon
       e2 = any(OperatorCall oc |
           oc.getTarget().(ConversionOperator).fromLibrary() and
           e1 = oc.getAnArgument() and
+          scope = e2 and
           isSuccessor = true
         )
     )

Original file line number	Diff line number	Diff line change
`@@ -117,6 +117,7 @@ private class LocalTaintExprStepConfiguration extends ControlFlowReachabilityCon`
`117`	`117`	`e2 = any(OperatorCall oc \|`
`118`	`118`	`oc.getTarget().(ConversionOperator).fromLibrary() and`
`119`	`119`	`e1 = oc.getAnArgument() and`
	`120`	`+ scope = e2 and`
`120`	`121`	`isSuccessor = true`
`121`	`122`	`)`
`122`	`123`	`)`