Merge pull request #10154 from cgocast/5.x

vimeo · Aug 27, 2023 · 77436b1 · 77436b1
2 parents 55fb269 + bb90299
commit 77436b1
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/stubs/Reflection.phpstub b/stubs/Reflection.phpstub
@@ -15,6 +15,7 @@ class ReflectionClass implements Reflector {
     /**
      * @param T|class-string<T>|interface-string<T>|trait-string|enum-string<T> $argument
      * @psalm-pure
+     * @psalm-taint-sink callable $argument
      */
     public function __construct($argument) {}
 
@@ -424,7 +425,7 @@ class ReflectionFunction extends ReflectionFunctionAbstract
 {
     /**
      * @param callable-string|Closure $function
-     *
+     * @psalm-taint-sink callable $function
      * @psalm-pure
      */
     public function __construct(callable $function) {}

diff --git a/tests/TaintTest.php b/tests/TaintTest.php
@@ -2489,6 +2489,20 @@ public static function getPrevious(string $s): string {
                     echo pg_escape_string($conn, $_GET["a"]);',
                 'error_message' => 'TaintedHtml',
             ],
+            'taintedReflectionClass' => [
+                'code' => '<?php
+                    $name = $_GET["name"];
+                    $reflector = new ReflectionClass($name);
+                    $reflector->newInstance();',
+                'error_message' => 'TaintedCallable',
+            ],
+            'taintedReflectionFunction' => [
+                'code' => '<?php
+                    $name = $_GET["name"];
+                    $function = new ReflectionFunction($name);
+                    $function->invoke();',
+                'error_message' => 'TaintedCallable',
+            ],
         ];
     }