From 394edbf3b5232c30dd3019f8c3a5dfadc310884f Mon Sep 17 00:00:00 2001 From: jvoisin Date: Wed, 19 Feb 2020 21:52:32 +0100 Subject: Improve a bit php8 compatibility PHP changed the way it exposes if a function has variadic arguments or not, hence why we need yet an other ifdef. --- src/sp_disabled_functions.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/sp_disabled_functions.c b/src/sp_disabled_functions.c index c088f20..9e64993 100644 --- a/src/sp_disabled_functions.c +++ b/src/sp_disabled_functions.c @@ -352,7 +352,12 @@ static void should_disable(zend_execute_data* execute_data, if (config_node->param || config_node->r_param || (config_node->pos != -1)) { if (!builtin_param && - execute_data->func->op_array.arg_info->is_variadic) { +#if PHP_VERSION_ID >= 80000 + ZEND_ARG_IS_VARIADIC(execute_data->func->op_array.arg_info) +#else + execute_data->func->op_array.arg_info->is_variadic +#endif + ){ sp_log_warn( "disable_function", "Snuffleupagus doesn't support variadic functions yet, sorry. " -- cgit v1.3