Skip to content

Commit a93d6a3

Browse files
author
edvraa
committed
Remove SafeConstructorTrackingConfig
1 parent 9e46ef3 commit a93d6a3

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

csharp/ql/src/Security Features/CWE-502/UnsafeDeserializationUntrustedInput.ql

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -39,14 +39,6 @@ where
3939
mc.getQualifier() = taintedTypeUsage.asExpr() and
4040
mc.getAnArgument() = deserializeCallArg.getNode().asExpr()
4141
)
42-
) and
43-
// exclude deserialization flows with safe instances (i.e. JavaScriptSerializer without resolver)
44-
not exists(
45-
SafeConstructorTrackingConfig safeConstructorTracking, DataFlow::Node safeCreation,
46-
DataFlow::Node safeTypeUsage
47-
|
48-
safeConstructorTracking.hasFlow(safeCreation, safeTypeUsage) and
49-
safeTypeUsage.asExpr().getParent() = deserializeCallArg.getNode().asExpr().getParent()
5042
)
5143
or
5244
// no type check needed - straightforward taint -> sink

0 commit comments

Comments
 (0)