Fix infinite recursion in ConstantArrayType::isCallable() via RecursionGuard

Author: MizouziE

src/Type/Constant/ConstantArrayType.php

@@ -42,6 +42,7 @@
 use PHPStan\Type\MixedType;
 use PHPStan\Type\NeverType;
 use PHPStan\Type\NullType;
+use PHPStan\Type\RecursionGuard;
 use PHPStan\Type\Traits\ArrayTypeTrait;
 use PHPStan\Type\Traits\NonObjectTypeTrait;
 use PHPStan\Type\Traits\UndecidedComparisonTypeTrait;
@@ -494,23 +495,32 @@ public function equals(Type $type): bool
 
 	public function isCallable(): TrinaryLogic
 	{
-		$hasNonExistentMethod = false;
-		$typeAndMethods = $this->doFindTypeAndMethodNames($hasNonExistentMethod);
-		if ($typeAndMethods === []) {
-			return TrinaryLogic::createNo();
-		}
+		$result = RecursionGuard::run($this, function (): TrinaryLogic {
+			$hasNonExistentMethod = false;
+			$typeAndMethods = $this->doFindTypeAndMethodNames($hasNonExistentMethod);
+			if ($typeAndMethods === []) {
+				return TrinaryLogic::createNo();
+			}
 
-		$results = array_map(
-			static fn (ConstantArrayTypeAndMethod $typeAndMethod): TrinaryLogic => $typeAndMethod->getCertainty(),
-			$typeAndMethods,
-		);
+			$results = array_map(
+				static fn (ConstantArrayTypeAndMethod $typeAndMethod): TrinaryLogic => $typeAndMethod->getCertainty(),
+				$typeAndMethods,
+			);
 
-		$result = TrinaryLogic::createYes()->and(...$results);
+			$result = TrinaryLogic::createYes()->and(...$results);
 
-		if ($hasNonExistentMethod) {
-			$result = $result->and(TrinaryLogic::createMaybe());
+			if ($hasNonExistentMethod) {
+				$result = $result->and(TrinaryLogic::createMaybe());
+			}
+
+			return $result;
+		});
+
+		if ($result instanceof ErrorType) {
+			return TrinaryLogic::createNo();
 		}
 
+
 		return $result;
 	}
 

tests/PHPStan/Analyser/AnalyserIntegrationTest.php

@@ -93,6 +93,14 @@ public function testInfiniteRecursionWithCallable(): void
 		$this->assertNoErrors($errors);
 	}
 
+	public function testConstantArrayCallableDoesNotCauseInfiniteRecursion(): void
+	{
+		// Previously caused infinite recursion / OOM via ConstantArrayType::isCallable()
+		// resolving getMethod() which triggered return type resolution that called isCallable() again
+		$errors = $this->runAnalyse(__DIR__ . '/data/bug-constant-array-callable-recursion.php');
+		$this->assertCount(3, $errors);
+	}
+
 	public function testClassThatExtendsUnknownClassIn3rdPartyPropertyTypeShouldNotCauseAutoloading(): void
 	{
 		// no error about PHPStan\Tests\Baz not being able to be autoloaded

tests/PHPStan/Analyser/data/bug-constant-array-callable-recursion.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace BugConstantArrayCallableRecursion;
+
+class Period
+{
+
+	/**
+	 * @return array{'BugConstantArrayCallableRecursion\Period', 'endIteration'}
+	 */
+	public function endIteration(): callable
+	{
+		return [self::class, 'endIteration'];
+	}
+
+}
+
+function test(): void
+{
+	$cb = ['BugConstantArrayCallableRecursion\Period', 'endIteration'];
+	is_callable($cb);
+}