archive/php-src
1
0
Fork 0
mirror of https://github.com/php/php-src.git synced 2025-08-16 05:58:45 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Convert mod user FINISH macro to static function

Browse source
This commit is contained in:
George Peter Banyard 2022-10-20 22:25:12 +01:00
parent 3917018806
commit 8e9fa2bd23
2 changed files with 81 additions and 31 deletions
Download patch file Download diff file Expand all files Collapse all files

80
ext/session/mod_user.c
View file

@ -47,31 +47,37 @@ static void ps_call_handler(zval *func, int argc, zval *argv, zval *retval)
#define PSF(a) PS(mod_user_names).ps_##a #define PSF(a) PS(mod_user_names).ps_##a
#define FINISH(return_value) \ static zend_result verify_bool_return_type_userland_calls(const zval* value)
if (Z_TYPE(return_value) != IS_UNDEF) { \ {
if (Z_TYPE(return_value) == IS_TRUE) { \ /* Exit or exception in userland call */
ret = SUCCESS; \ if (Z_TYPE_P(value) == IS_UNDEF) {
} else if (Z_TYPE(return_value) == IS_FALSE) { \ return FAILURE;
ret = FAILURE; \ }
} else if ((Z_TYPE(return_value) == IS_LONG) && (Z_LVAL(return_value) == -1)) { \ if (Z_TYPE_P(value) == IS_TRUE) {
if (!EG(exception)) { \ return SUCCESS;
php_error_docref(NULL, E_DEPRECATED, "Session callback must have a return value of type bool, %s returned", zend_zval_type_name(&return_value)); \ }
} \ if (Z_TYPE_P(value) == IS_FALSE) {
ret = FAILURE; \ return FAILURE;
} else if ((Z_TYPE(return_value) == IS_LONG) && (Z_LVAL(return_value) == 0)) { \ }
if (!EG(exception)) { \ if ((Z_TYPE_P(value) == IS_LONG) && (Z_LVAL_P(value) == -1)) {
php_error_docref(NULL, E_DEPRECATED, "Session callback must have a return value of type bool, %s returned", zend_zval_type_name(&return_value)); \ /* TODO Why are exception cheked? */
} \ if (!EG(exception)) {
ret = SUCCESS; \ php_error_docref(NULL, E_DEPRECATED, "Session callback must have a return value of type bool, %s returned", zend_zval_type_name(value));
} else { \ }
if (!EG(exception)) { \ return FAILURE;
zend_type_error("Session callback must have a return value of type bool, %s returned", zend_zval_type_name(&return_value)); \ }
} \ if ((Z_TYPE_P(value) == IS_LONG) && (Z_LVAL_P(value) == 0)) {
ret = FAILURE; \ /* TODO Why are exception cheked? */
zval_ptr_dtor(&return_value); \ if (!EG(exception)) {
} \ php_error_docref(NULL, E_DEPRECATED, "Session callback must have a return value of type bool, %s returned", zend_zval_type_name(value));
} \ }
return ret return SUCCESS;
}
if (!EG(exception)) {
zend_type_error("Session callback must have a return value of type bool, %s returned", zend_zval_type_name(value)); \
}
return FAILURE;
}
PS_OPEN_FUNC(user) PS_OPEN_FUNC(user)
{ {
@ -96,7 +102,9 @@ PS_OPEN_FUNC(user)
PS(mod_user_implemented) = 1; PS(mod_user_implemented) = 1;
FINISH(retval); ret = verify_bool_return_type_userland_calls(&retval);
zval_ptr_dtor(&retval);
return ret;
} }
PS_CLOSE_FUNC(user) PS_CLOSE_FUNC(user)
@ -127,7 +135,9 @@ PS_CLOSE_FUNC(user)
zend_bailout(); zend_bailout();
} }
FINISH(retval); ret = verify_bool_return_type_userland_calls(&retval);
zval_ptr_dtor(&retval);
return ret;
} }
PS_READ_FUNC(user) PS_READ_FUNC(user)
@ -166,7 +176,9 @@ PS_WRITE_FUNC(user)
ps_call_handler(&PSF(write), 2, args, &retval); ps_call_handler(&PSF(write), 2, args, &retval);
FINISH(retval); ret = verify_bool_return_type_userland_calls(&retval);
zval_ptr_dtor(&retval);
return ret;
} }
PS_DESTROY_FUNC(user) PS_DESTROY_FUNC(user)
@ -181,7 +193,9 @@ PS_DESTROY_FUNC(user)
ps_call_handler(&PSF(destroy), 1, args, &retval); ps_call_handler(&PSF(destroy), 1, args, &retval);
FINISH(retval); ret = verify_bool_return_type_userland_calls(&retval);
zval_ptr_dtor(&retval);
return ret;
} }
PS_GC_FUNC(user) PS_GC_FUNC(user)
@ -250,7 +264,9 @@ PS_VALIDATE_SID_FUNC(user)
ps_call_handler(&PSF(validate_sid), 1, args, &retval); ps_call_handler(&PSF(validate_sid), 1, args, &retval);
FINISH(retval); ret = verify_bool_return_type_userland_calls(&retval);
zval_ptr_dtor(&retval);
return ret;
} }
/* dummy function defined by PS_MOD */ /* dummy function defined by PS_MOD */
@ -273,5 +289,7 @@ PS_UPDATE_TIMESTAMP_FUNC(user)
ps_call_handler(&PSF(write), 2, args, &retval); ps_call_handler(&PSF(write), 2, args, &retval);
} }
FINISH(retval); ret = verify_bool_return_type_userland_calls(&retval);
zval_ptr_dtor(&retval);
return ret;
} }

32
ext/session/tests/user_session_module/session_set_save_handler_type_error2.phpt Normal file
View file

@ -0,0 +1,32 @@
--TEST--
Test session_set_save_handler() function: Incorrect bool returns
--EXTENSIONS--
session
--FILE--
<?php
$validCallback = function () { return true; };
$nullCallback = function () { return; };
$oneCallback = function () { return 1; };
ob_start();
try {
$ret = session_set_save_handler($nullCallback, $validCallback, $validCallback, $validCallback, $validCallback, $validCallback);
session_start();
} catch (TypeError $exception) {
echo $exception->getMessage() . "\n";
}
try {
$ret = session_set_save_handler($oneCallback, $validCallback, $validCallback, $validCallback, $validCallback, $validCallback);
session_start();
} catch (TypeError $exception) {
echo $exception->getMessage() . "\n";
}
ob_end_flush();
?>
--EXPECT--
Session callback must have a return value of type bool, null returned
Session callback must have a return value of type bool, int returned