Skip to content

Commit aad55ff

Browse files
authored
Merge pull request #7279 from hvitved/csharp/json-net-fix-summaries
C#: Fix `Newtonsoft.Json.JsonSerializer.{Deserialize,Serialize}` summaries
2 parents 75f9a94 + 93e291c commit aad55ff

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

csharp/ql/lib/semmle/code/csharp/frameworks/JsonNET.qll

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -153,14 +153,14 @@ module JsonNET {
153153
// Serialize
154154
c = this.getSerializeMethod() and
155155
preservesValue = false and
156-
source = any(CallableFlowSourceArg arg | arg.getArgumentIndex() = 0) and
157-
sink = any(CallableFlowSinkArg arg | arg.getArgumentIndex() = 1)
156+
source = any(CallableFlowSourceArg arg | arg.getArgumentIndex() = 1) and
157+
sink = any(CallableFlowSinkArg arg | arg.getArgumentIndex() = 0)
158158
or
159159
// Deserialize
160160
c = this.getDeserializeMethod() and
161161
preservesValue = false and
162162
source = any(CallableFlowSourceArg arg | arg.getArgumentIndex() = 0) and
163-
sink = any(CallableFlowSinkArg arg | arg.getArgumentIndex() = 1)
163+
sink instanceof CallableFlowSinkReturn
164164
}
165165
}
166166

0 commit comments

Comments
 (0)