Reverted ad4533fdbabcc3e545277e30023b2fdce16297a0

The E_ERROR to E_RECOVERABLE_ERROR should be readded with the
proper tests.

diff --git a/ext/session/mod_user.c b/ext/session/mod_user.c
index 34a5a2134630e5780f2c78e22ed8d169375619c5..1196d867af59ba95969d4c12ecd2b2ed79f3c0e5 100644 (file)
--- a/ext/session/mod_user.c
+++ b/ext/session/mod_user.c
@@ -191,12 +191,12 @@ PS_CREATE_SID_FUNC(user)
                        }
                        zval_ptr_dtor(&retval);
                } else {
-                       php_error_docref(NULL, E_RECOVERABLE_ERROR, "No session id returned by function");
+                       php_error_docref(NULL, E_ERROR, "No session id returned by function");
                        return NULL;
                }
 
                if (!id) {
-                       php_error_docref(NULL, E_RECOVERABLE_ERROR, "Session id must be a string");
+                       php_error_docref(NULL, E_ERROR, "Session id must be a string");
                        return NULL;
                }
 

diff --git a/ext/session/mod_user_class.c b/ext/session/mod_user_class.c
index d06902ce27796ebe989c7d1a510a869c01cc4fc8..82aea1485423a86c40b1e3b3e46a7a613958fd9c 100644 (file)
--- a/ext/session/mod_user_class.c
+++ b/ext/session/mod_user_class.c
@@ -23,7 +23,7 @@
 
 #define PS_SANITY_CHECK                                                \
        if (PS(default_mod) == NULL) {                          \
-               php_error_docref(NULL, E_RECOVERABLE_ERROR, "Cannot call default session handler"); \
+               php_error_docref(NULL, E_CORE_ERROR, "Cannot call default session handler"); \
                RETURN_FALSE;                                           \
        }
 

diff --git a/ext/session/session.c b/ext/session/session.c
index cf4af113f32c425e665f17b36f57fc2d5b88653f..ef17b86bad31a0d752d104cf9e525990ab214b43 100644 (file)
--- a/ext/session/session.c
+++ b/ext/session/session.c
@@ -339,7 +339,7 @@ PHPAPI zend_string *php_session_create_id(PS_CREATE_SID_ARGS) /* {{{ */
                case PS_HASH_FUNC_OTHER:
                        if (!PS(hash_ops)) {
                                efree(buf);
-                               php_error_docref(NULL, E_RECOVERABLE_ERROR, "Invalid session hash function");
+                               php_error_docref(NULL, E_ERROR, "Invalid session hash function");
                                return NULL;
                        }
 
@@ -351,7 +351,7 @@ PHPAPI zend_string *php_session_create_id(PS_CREATE_SID_ARGS) /* {{{ */
 #endif /* HAVE_HASH_EXT */
                default:
                        efree(buf);
-                       php_error_docref(NULL, E_RECOVERABLE_ERROR, "Invalid session hash function");
+                       php_error_docref(NULL, E_ERROR, "Invalid session hash function");
                        return NULL;
        }
        efree(buf);
@@ -480,7 +480,7 @@ static void php_session_initialize(void) /* {{{ */
        zend_string *val = NULL;
 
        if (!PS(mod)) {
-               php_error_docref(NULL,  E_RECOVERABLE_ERROR, "No storage module chosen - failed to initialize session");
+               php_error_docref(NULL, E_ERROR, "No storage module chosen - failed to initialize session");
                return;
        }
 
@@ -488,7 +488,7 @@ static void php_session_initialize(void) /* {{{ */
        if (PS(mod)->s_open(&PS(mod_data), PS(save_path), PS(session_name)) == FAILURE
                /* || PS(mod_data) == NULL */ /* FIXME: open must set valid PS(mod_data) with success */
        ) {
-               php_error_docref(NULL, E_RECOVERABLE_ERROR, "Failed to initialize storage module: %s (path: %s)", PS(mod)->s_name, PS(save_path));
+               php_error_docref(NULL, E_ERROR, "Failed to initialize storage module: %s (path: %s)", PS(mod)->s_name, PS(save_path));
                return;
        }
 
@@ -496,7 +496,7 @@ static void php_session_initialize(void) /* {{{ */
        if (!PS(id)) {
                PS(id) = PS(mod)->s_create_sid(&PS(mod_data));
                if (!PS(id)) {
-                       php_error_docref(NULL, E_RECOVERABLE_ERROR, "Failed to create session ID: %s (path: %s)", PS(mod)->s_name, PS(save_path));
+                       php_error_docref(NULL, E_ERROR, "Failed to create session ID: %s (path: %s)", PS(mod)->s_name, PS(save_path));
                        return;
                }
                if (PS(use_cookies)) {
@@ -600,7 +600,7 @@ static PHP_INI_MH(OnUpdateSaveHandler) /* {{{ */
                int err_type;
 
                if (stage == ZEND_INI_STAGE_RUNTIME) {
-                       err_type = E_RECOVERABLE_ERROR;
+                       err_type = E_WARNING;
                } else {
                        err_type = E_ERROR;
                }
@@ -630,7 +630,7 @@ static PHP_INI_MH(OnUpdateSerializer) /* {{{ */
                int err_type;
 
                if (stage == ZEND_INI_STAGE_RUNTIME) {
-                       err_type = E_RECOVERABLE_ERROR;
+                       err_type = E_WARNING;
                } else {
                        err_type = E_ERROR;
                }
@@ -699,7 +699,7 @@ static PHP_INI_MH(OnUpdateName) /* {{{ */
                int err_type;
 
                if (stage == ZEND_INI_STAGE_RUNTIME || stage == ZEND_INI_STAGE_ACTIVATE || stage == ZEND_INI_STAGE_STARTUP) {
-                       err_type = E_RECOVERABLE_ERROR;
+                       err_type = E_WARNING;
                } else {
                        err_type = E_ERROR;
                }
@@ -1829,7 +1829,7 @@ static PHP_FUNCTION(session_set_save_handler)
                                add_next_index_zval(&PS(mod_user_names).names[i], obj);
                                add_next_index_str(&PS(mod_user_names).names[i], zend_string_copy(func_name));
                        } else {
-                               php_error_docref(NULL, E_RECOVERABLE_ERROR, "Session handler's function table is corrupt");
+                               php_error_docref(NULL, E_ERROR, "Session handler's function table is corrupt");
                                RETURN_FALSE;
                        }
 

diff --git a/ext/session/tests/bug60860.phpt b/ext/session/tests/bug60860.phpt
index 194239d4c1ddbaa478b0f04d8afa727a41ec784d..83185862f01f02e92189f6d18e674dff0c17b5dd 100644 (file)
--- a/ext/session/tests/bug60860.phpt
+++ b/ext/session/tests/bug60860.phpt
@@ -14,4 +14,4 @@ echo "ok\n";
 --EXPECTF--
 Warning: session_start(): user session functions not defined in %s on line 3
 
-Catchable fatal error: session_start(): Failed to initialize storage module: user (path:%s) in %s on line 3
+Fatal error: session_start(): Failed to initialize storage module: user (path:%s) in %s on line 3

diff --git a/ext/session/tests/bug66481.phpt b/ext/session/tests/bug66481.phpt
index 27171da8f307a5a06de86532ba188b8e2843c56a..5525ae8a38dffcdd6dd7c1efc116b94ed132d9c7 100644 (file)
--- a/ext/session/tests/bug66481.phpt
+++ b/ext/session/tests/bug66481.phpt
@@ -11,8 +11,8 @@ session.name=
 var_dump(session_name("foo"));
 var_dump(session_name("bar"));
 --EXPECTF--
-PHP Catchable fatal error:  PHP Startup: session.name cannot be a numeric or empty '' in Unknown on line 0
+PHP Warning:  PHP Startup: session.name cannot be a numeric or empty '' in Unknown on line 0
 
-Catchable fatal error: PHP Startup: session.name cannot be a numeric or empty '' in Unknown on line 0
+Warning: PHP Startup: session.name cannot be a numeric or empty '' in Unknown on line 0
 string(9) "PHPSESSID"
 string(3) "foo"

diff --git a/ext/session/tests/bug67972.phpt b/ext/session/tests/bug67972.phpt
index c4bcc1a0868b13a88004526051f7bebe304755dc..63ed3a95b8978df9a8daf24a7b9cea184019d03d 100644 (file)
--- a/ext/session/tests/bug67972.phpt
+++ b/ext/session/tests/bug67972.phpt
@@ -7,4 +7,4 @@ Bug #67972: SessionHandler Invalid memory read create_sid()
 
 (new SessionHandler)->create_sid();
 --EXPECTF--
-Catchable fatal error: SessionHandler::create_sid(): Cannot call default session handler in %s on line %d
+Fatal error: SessionHandler::create_sid(): Cannot call default session handler in %s on line %d

diff --git a/ext/session/tests/session_module_name_variation3.phpt b/ext/session/tests/session_module_name_variation3.phpt
index aabb8baa6f55d88398119838ff043819a1be24c9..de49195fe8445e975f04f06f00a61fd103e96762 100644 (file)
--- a/ext/session/tests/session_module_name_variation3.phpt
+++ b/ext/session/tests/session_module_name_variation3.phpt
@@ -48,4 +48,4 @@ Stack trace:
 #2 {main}
   thrown in %s on line %d
 
-Catchable fatal error: session_start(): Failed to initialize storage module: %s in %s%esession_module_name_variation3.php on line %d
+Fatal error: session_start(): Failed to initialize storage module: %s in %s%esession_module_name_variation3.php on line %d

diff --git a/ext/session/tests/session_name_error.phpt b/ext/session/tests/session_name_error.phpt
index cd7e364882d14ff3cc8a0788bba964e0b41ae459..9f0101d98b5de8105a43f248b4fa78a0b8ff5eb3 100644 (file)
--- a/ext/session/tests/session_name_error.phpt
+++ b/ext/session/tests/session_name_error.phpt
@@ -8,21 +8,6 @@ session.name=PHPSESSID
 --FILE--
 <?php
 
-function error_handler($errno, $errstr, $errfile, $errline)
-{
-       if ($errno & E_NOTICE) {
-               return ture; // Ignore notices
-       }
-    if ($errno & E_RECOVERABLE_ERROR) {
-        // Handle E_REVOCERABLE_ERROR
-        echo "\nE_RECOVERABLE_ERROR: {$errstr} in {$errfile} on line {$errline}\n";
-        return true; // Continue execution
-    }
-       return false;
-}
-
-set_error_handler('error_handler');
-
 ob_start();
 
 /* 
@@ -114,103 +99,103 @@ ob_end_flush();
 
 -- Iteration 1 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '0' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '0' in %s on line %d
 int(0)
 string(9) "PHPSESSID"
 
 -- Iteration 2 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '1' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '1' in %s on line %d
 int(1)
 string(9) "PHPSESSID"
 
 -- Iteration 3 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '12345' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '12345' in %s on line %d
 int(12345)
 string(9) "PHPSESSID"
 
 -- Iteration 4 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '-2345' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '-2345' in %s on line %d
 int(-2345)
 string(9) "PHPSESSID"
 
 -- Iteration 5 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '10.5' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '10.5' in %s on line %d
 float(10.5)
 string(9) "PHPSESSID"
 
 -- Iteration 6 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '-10.5' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '-10.5' in %s on line %d
 float(-10.5)
 string(9) "PHPSESSID"
 
 -- Iteration 7 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '123456789000' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '123456789000' in %s on line %d
 float(123456789000)
 string(9) "PHPSESSID"
 
 -- Iteration 8 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '1.23456789E-9' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '1.23456789E-9' in %s on line %d
 float(1.23456789E-9)
 string(9) "PHPSESSID"
 
 -- Iteration 9 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '0.5' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '0.5' in %s on line %d
 float(0.5)
 string(9) "PHPSESSID"
 
 -- Iteration 10 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
 NULL
 string(9) "PHPSESSID"
 
 -- Iteration 11 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
 NULL
 string(9) "PHPSESSID"
 
 -- Iteration 12 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '1' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '1' in %s on line %d
 bool(true)
 string(9) "PHPSESSID"
 
 -- Iteration 13 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
 bool(false)
 string(9) "PHPSESSID"
 
 -- Iteration 14 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '1' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '1' in %s on line %d
 bool(true)
 string(9) "PHPSESSID"
 
 -- Iteration 15 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
 bool(false)
 string(9) "PHPSESSID"
 
 -- Iteration 16 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
 string(0) ""
 string(9) "PHPSESSID"
 
 -- Iteration 17 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
 string(0) ""
 string(9) "PHPSESSID"
 
@@ -233,13 +218,13 @@ string(12) "Hello World!"
 
 -- Iteration 22 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
 NULL
 string(12) "Hello World!"
 
 -- Iteration 23 --
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
 NULL
 string(12) "Hello World!"
 

diff --git a/ext/session/tests/session_name_variation1.phpt b/ext/session/tests/session_name_variation1.phpt
index c00ffbfc86d696a8749886694f7a98df5b8a8200..b0de3ee361a7dc51ddd302248873d883f0384d2d 100644 (file)
--- a/ext/session/tests/session_name_variation1.phpt
+++ b/ext/session/tests/session_name_variation1.phpt
@@ -7,20 +7,6 @@ session.name=PHPSESSID
 <?php include('skipif.inc'); ?>
 --FILE--
 <?php
-function error_handler($errno, $errstr, $errfile, $errline)
-{
-    if ($errno & E_NOTICE) {
-        return ture; // Ignore notices
-    }
-    if ($errno & E_RECOVERABLE_ERROR) {
-        // Handle E_REVOCERABLE_ERROR
-        echo "\nE_RECOVERABLE_ERROR: {$errstr} in {$errfile} on line {$errline}\n";
-        return true; // Continue execution
-    }
-    return false;
-}
-
-set_error_handler('error_handler');
 
 ob_start();
 
@@ -66,7 +52,7 @@ string(1) "   "
 bool(true)
 string(1) "    "
 
-E_RECOVERABLE_ERROR: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
+Warning: session_name(): session.name cannot be a numeric or empty '' in %s on line %d
 string(1) "    "
 bool(true)
 string(1) "    "

diff --git a/ext/session/tests/session_save_path_variation4.phpt b/ext/session/tests/session_save_path_variation4.phpt
index 0783938eae43f25c304f8450f1b5af56252fda6d..80db6caf7dcb1d28dd3627156cbeaaf7169b60f5 100644 (file)
--- a/ext/session/tests/session_save_path_variation4.phpt
+++ b/ext/session/tests/session_save_path_variation4.phpt
@@ -56,5 +56,5 @@ string(0) ""
 
 Warning: session_start(): open_basedir restriction in effect. File(%s) is not within the allowed path(s): (.) in %s on line %d
 
-Catchable fatal error: session_start(): Failed to initialize storage module: files (path: ) in %s on line %d
+Fatal error: session_start(): Failed to initialize storage module: files (path: ) in %s on line %d
 

diff --git a/ext/session/tests/session_set_save_handler_class_014.phpt b/ext/session/tests/session_set_save_handler_class_014.phpt
index 664590769b91dc33b88b2026b7b5f22300566a61..ea62beb0ff57eff2aec7ed93214f14229839372c 100644 (file)
--- a/ext/session/tests/session_set_save_handler_class_014.phpt
+++ b/ext/session/tests/session_set_save_handler_class_014.phpt
@@ -27,4 +27,4 @@ session_start();
 --EXPECTF--
 *** Testing session_set_save_handler() : calling default handler when save_handler=user ***
 
-Catchable fatal error: SessionHandler::open(): Cannot call default session handler in %s on line %d
+Fatal error: SessionHandler::open(): Cannot call default session handler in %s on line %d

diff --git a/ext/session/tests/session_set_save_handler_error3.phpt b/ext/session/tests/session_set_save_handler_error3.phpt
index 345ebe9583bfd4b41c2c35872a1770d73c0087b9..fdf306a4a52b12e31da092d6d829bd4120a8e728 100644 (file)
--- a/ext/session/tests/session_set_save_handler_error3.phpt
+++ b/ext/session/tests/session_set_save_handler_error3.phpt
@@ -41,4 +41,4 @@ Stack trace:
 #2 {main}
   thrown in %s on line %d
 
-Catchable fatal error: session_start(): Failed to initialize storage module: %s in %ssession_set_save_handler_error3.php on line %d
+Fatal error: session_start(): Failed to initialize storage module: %s in %ssession_set_save_handler_error3.php on line %d

diff --git a/ext/session/tests/session_set_save_handler_sid_002.phpt b/ext/session/tests/session_set_save_handler_sid_002.phpt
index f9edb82b54905d9a8d465d99d09087a85c6d0ed2..f9a72aebcaf8e7e14d66451e297fff01bbd85858 100644 (file)
--- a/ext/session/tests/session_set_save_handler_sid_002.phpt
+++ b/ext/session/tests/session_set_save_handler_sid_002.phpt
@@ -74,4 +74,4 @@ session_unset();
 --EXPECTF--
 *** Testing session_set_save_handler() function: create_sid ***
 
-Catchable fatal error: session_start(): Session id must be a string in %s on line %d
+Fatal error: session_start(): Session id must be a string in %s on line %d