diff options
| author | Ben Fuhrmannek | 2022-01-07 18:01:50 +0100 |
|---|---|---|
| committer | Ben Fuhrmannek | 2022-01-07 18:01:50 +0100 |
| commit | c102fd3535cbed3d8a22d77306dedf2caf7c8674 (patch) | |
| tree | 9092943cea23cf6bd188e17acc2492eb6877e22e /src | |
| parent | 4b5afd0148cef6c845a37aff68e1fbac8f5653d7 (diff) | |
fixed invalid return value
Diffstat (limited to '')
| -rw-r--r-- | src/sp_disabled_functions.c | 2 | ||||
| -rw-r--r-- | src/sp_execute.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/sp_disabled_functions.c b/src/sp_disabled_functions.c index c0c642b..0ff859c 100644 --- a/src/sp_disabled_functions.c +++ b/src/sp_disabled_functions.c | |||
| @@ -469,7 +469,7 @@ static void should_drop_on_ret(const zval* return_value, | |||
| 469 | 469 | ||
| 470 | ZEND_FUNCTION(check_disabled_function) { | 470 | ZEND_FUNCTION(check_disabled_function) { |
| 471 | zif_handler orig_handler; | 471 | zif_handler orig_handler; |
| 472 | const char* current_function_name = get_active_function_name(TSRMLS_C); | 472 | const char* current_function_name = get_active_function_name(); |
| 473 | 473 | ||
| 474 | should_disable_ht(execute_data, current_function_name, NULL, NULL, SPCFG(disabled_functions_reg).disabled_functions, SPCFG(disabled_functions_hooked)); | 474 | should_disable_ht(execute_data, current_function_name, NULL, NULL, SPCFG(disabled_functions_reg).disabled_functions, SPCFG(disabled_functions_hooked)); |
| 475 | 475 | ||
diff --git a/src/sp_execute.c b/src/sp_execute.c index aadd145..f1ed8d0 100644 --- a/src/sp_execute.c +++ b/src/sp_execute.c | |||
| @@ -190,7 +190,7 @@ static inline void sp_execute_handler(INTERNAL_FUNCTION_PARAMETERS, bool interna | |||
| 190 | // execute_data, so we need to use a local variable to be able to match on | 190 | // execute_data, so we need to use a local variable to be able to match on |
| 191 | // it later. | 191 | // it later. |
| 192 | zval ret_val; | 192 | zval ret_val; |
| 193 | if (EX(return_value) == NULL) { | 193 | if (EX(return_value) == NULL && return_value == NULL) { |
| 194 | memset(&ret_val, 0, sizeof(ret_val)); | 194 | memset(&ret_val, 0, sizeof(ret_val)); |
| 195 | return_value = EX(return_value) = &ret_val; | 195 | return_value = EX(return_value) = &ret_val; |
| 196 | } | 196 | } |
