Swift: remove ArrayContent in UnsafeJsEvalQuery

Robert Marsh · Robert Marsh · commit 43ca192ceba9 · 2023-09-13T18:26:06.000Z
diff --git a/swift/ql/lib/codeql/swift/security/UnsafeJsEvalQuery.qll b/swift/ql/lib/codeql/swift/security/UnsafeJsEvalQuery.qll
@@ -30,10 +30,7 @@ module UnsafeJsEvalConfig implements DataFlow::ConfigSig {
       or
       isAdditionalFlowStep(node, _)
     ) and
-    (
-      c.getAReadContent() instanceof DataFlow::Content::ArrayContent or
-      c.getAReadContent() instanceof DataFlow::Content::CollectionContent
-    )
+    c.getAReadContent() instanceof DataFlow::Content::CollectionContent
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -30,10 +30,7 @@ module UnsafeJsEvalConfig implements DataFlow::ConfigSig {`
`30`	`30`	`or`
`31`	`31`	`isAdditionalFlowStep(node, _)`
`32`	`32`	`) and`
`33`		`- (`
`34`		`- c.getAReadContent() instanceof DataFlow::Content::ArrayContent or`
`35`		`- c.getAReadContent() instanceof DataFlow::Content::CollectionContent`
`36`		`- )`
	`33`	`+ c.getAReadContent() instanceof DataFlow::Content::CollectionContent`
`37`	`34`	`}`
`38`	`35`	`}`
`39`	`36`