diff --git a/phpseclib/Math/BigInteger.php b/phpseclib/Math/BigInteger.php index 99690817..3f4dc2ed 100644 --- a/phpseclib/Math/BigInteger.php +++ b/phpseclib/Math/BigInteger.php @@ -136,16 +136,6 @@ class BigInteger implements \JsonSerializable private static function initialize_static_variables() { if (!isset(self::$mainEngine)) { - // see https://github.com/php/php-src/issues/11917 - if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN' && function_exists('opcache_get_status') && !defined('PHPSECLIB_ALLOW_JIT')) { - $status = opcache_get_status(); - if ($status && isset($status['jit']) && $status['jit']['enabled'] && $status['jit']['on']) { - throw new UnexpectedValueException( - 'JIT on Windows is not currently supported' - ); - } - } - $engines = [ ['GMP', ['DefaultEngine']], ['PHP64', ['OpenSSL']], @@ -154,13 +144,16 @@ class BigInteger implements \JsonSerializable ['PHP64', ['DefaultEngine']], ['PHP32', ['DefaultEngine']] ]; + foreach ($engines as $engine) { try { self::setEngine($engine[0], $engine[1]); - break; + return; } catch (\Exception $e) { } } + + throw new UnexpectedValueException('No valid BigInteger found. This is only possible when JIT is enabled on Windows and neither the GMP or BCMath extensions are available so either disable JIT or install GMP / BCMath'); } } diff --git a/phpseclib/Math/BigInteger/Engines/PHP.php b/phpseclib/Math/BigInteger/Engines/PHP.php index ab9bdc99..4c30da56 100644 --- a/phpseclib/Math/BigInteger/Engines/PHP.php +++ b/phpseclib/Math/BigInteger/Engines/PHP.php @@ -1326,4 +1326,18 @@ abstract class PHP extends Engine return array_reverse($vals); } + + /** + * @return bool + */ + protected static function testJITOnWindows() + { + if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN' && function_exists('opcache_get_status') && !defined('PHPSECLIB_ALLOW_JIT')) { + $status = opcache_get_status(); + if ($status && isset($status['jit']) && $status['jit']['enabled'] && $status['jit']['on']) { + return true; + } + } + return false; + } } diff --git a/phpseclib/Math/BigInteger/Engines/PHP32.php b/phpseclib/Math/BigInteger/Engines/PHP32.php index 18f78cdb..3a775e7d 100644 --- a/phpseclib/Math/BigInteger/Engines/PHP32.php +++ b/phpseclib/Math/BigInteger/Engines/PHP32.php @@ -102,7 +102,7 @@ class PHP32 extends PHP */ public static function isValidEngine() { - return PHP_INT_SIZE >= 4; + return PHP_INT_SIZE >= 4 && !self::testJITOnWindows(); } /** diff --git a/phpseclib/Math/BigInteger/Engines/PHP64.php b/phpseclib/Math/BigInteger/Engines/PHP64.php index ca11c08d..70a2e173 100644 --- a/phpseclib/Math/BigInteger/Engines/PHP64.php +++ b/phpseclib/Math/BigInteger/Engines/PHP64.php @@ -103,7 +103,7 @@ class PHP64 extends PHP */ public static function isValidEngine() { - return PHP_INT_SIZE >= 8; + return PHP_INT_SIZE >= 8 && !self::testJITOnWindows(); } /**