Correctly specify regex.

AdnaneKhan · JarLob · web-flow · commit f4f919635a7e · 2025-07-08T10:17:29.000-04:00
Co-authored-by: Jaroslav Lobačevski &lt;jarlob@github.com&gt;
diff --git a/actions/ql/lib/codeql/actions/security/ArtifactPoisoningQuery.qll b/actions/ql/lib/codeql/actions/security/ArtifactPoisoningQuery.qll
@@ -264,7 +264,7 @@ class ArtifactPoisoningSink extends DataFlow::Node {
     download.getAFollowingStep() = poisonable and
     // excluding artifacts downloaded to /tmp and runner.tmp
     not download.getPath().regexpMatch("^/tmp.*") and
-    not download.getPath().regexpMatch("^\\${{\\s?runner.temp\\s?}}.*") and
+    not download.getPath().regexpMatch("^\\$\\{\\{\\s?runner\\.temp\\s?}}.*") and
     (
       poisonable.(Run).getScript() = this.asExpr() and
       (

Original file line number	Diff line number	Diff line change
`@@ -264,7 +264,7 @@ class ArtifactPoisoningSink extends DataFlow::Node {`
`264`	`264`	`download.getAFollowingStep() = poisonable and`
`265`	`265`	`// excluding artifacts downloaded to /tmp and runner.tmp`
`266`	`266`	`not download.getPath().regexpMatch("^/tmp.*") and`
`267`		`- not download.getPath().regexpMatch("^\\${{\\s?runner.temp\\s?}}.*") and`
	`267`	`+ not download.getPath().regexpMatch("^\\$\\{\\{\\s?runner\\.temp\\s?}}.*") and`
`268`	`268`	`(`
`269`	`269`	`poisonable.(Run).getScript() = this.asExpr() and`
`270`	`270`	`(`