Remove internal usage of SplFileInfo::_bad_state_ex() method

Use standard VM handling instead

Deprecate the method

UPGRADING

@@ -149,6 +149,8 @@ PHP 8.2 UPGRADE NOTES
     separate, built-in implementations of all of them; for example, UUencoded
     data can be handled using convert_uuencode/convert_uudecode.
 
+- SPL:
+  . The SplFileInfo::_bad_state_ex() internal method has been deprecated.
 ========================================
 5. Changed Functions
 ========================================

ext/spl/spl_directory.c

@@ -693,11 +693,8 @@ zend_function *spl_filesystem_object_get_method_check(zend_object **object, zend
 	spl_filesystem_object *fsobj = spl_filesystem_from_obj(*object);
 
 	if (fsobj->u.dir.dirp == NULL && fsobj->orig_path == NULL) {
-		zend_function *func;
-		zend_string *tmp = zend_string_init("_bad_state_ex", sizeof("_bad_state_ex") - 1, 0);
-		func = zend_std_get_method(object, tmp, NULL);
-		zend_string_release_ex(tmp, 0);
-		return func;
+		zend_throw_error(NULL, "The parent constructor was not called: the object is in an invalid state");
+		return NULL;
 	}
 
 	return zend_std_get_method(object, method, key);
@@ -1403,6 +1400,7 @@ PHP_METHOD(SplFileInfo, _bad_state_ex)
 		RETURN_THROWS();
 	}
 	zend_throw_error(NULL, "The parent constructor was not called: the object is in an invalid state");
+	RETURN_THROWS();
 }
 /* }}} */
 

ext/spl/spl_directory.stub.php

@@ -97,7 +97,10 @@ public function __toString(): string {}
     /** @tentative-return-type */
     public function __debugInfo(): array {}
 
-    /** @tentative-return-type */
+    /**
+     * @deprecated
+     * @tentative-return-type
+     */
     final public function _bad_state_ex(): void {}
 }
 

ext/spl/tests/gh8318.phpt

@@ -0,0 +1,41 @@
+--TEST--
+GH-8318 (SplFileObject useless call to an internal method for erroring)
+--FILE--
+<?php
+class bug8318 extends \SplFileObject
+{
+    public function __construct()
+    {
+    }
+
+    public function fpassthru(): int
+    {
+        return 0;
+    }
+}
+
+$cl = new bug8318;
+try {
+    $cl->fpassthru();
+} catch (\Error $e) {
+    var_dump($e);
+}
+?>
+--EXPECTF--
+object(Error)#2 (7) {
+  ["message":protected]=>
+  string(72) "The parent constructor was not called: the object is in an invalid state"
+  ["string":"Error":private]=>
+  string(0) ""
+  ["code":protected]=>
+  int(0)
+  ["file":protected]=>
+  string(50) "%s"
+  ["line":protected]=>
+  int(16)
+  ["trace":"Error":private]=>
+  array(0) {
+  }
+  ["previous":"Error":private]=>
+  NULL
+}