Skip to content

Commit fd4d8e2

Browse files
author
edvraa
committed
Use HasFlow instead HasFlowPath
1 parent a0942e0 commit fd4d8e2

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

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

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,11 @@ where
5050
// JsonConvert static method call, but with additional unsafe typename tracking
5151
exists(
5252
JsonConvertTrackingConfig taintTrackingJsonConvert, TypeNameTrackingConfig typenameTracking,
53-
DataFlow::PathNode settingsCallArg
53+
DataFlow::Node settingsCallArg
5454
|
5555
taintTrackingJsonConvert.hasFlowPath(userInput, deserializeCallArg) and
56-
typenameTracking.hasFlowPath(_, settingsCallArg) and
57-
deserializeCallArg.getNode().asExpr().getParent() =
58-
settingsCallArg.getNode().asExpr().getParent()
56+
typenameTracking.hasFlow(_, settingsCallArg) and
57+
deserializeCallArg.getNode().asExpr().getParent() = settingsCallArg.asExpr().getParent()
5958
)
6059
select deserializeCallArg, userInput, deserializeCallArg, "$@ flows to unsafe deserializer.",
6160
userInput, "User-provided data"

0 commit comments

Comments
 (0)