diff options
| author | Thibault "bui" Koechlin | 2017-10-18 13:17:46 +0200 |
|---|---|---|
| committer | jvoisin | 2017-10-18 13:17:46 +0200 |
| commit | 8f94a1179e9cf9d96f18ec6c851dddbecd54ea6f (patch) | |
| tree | 970a0cd9f700935e4bd16e73dc630dd3e36554ab /src/sp_disabled_functions.c | |
| parent | da49f804ad5ae283c10ad757cd39bcc212b66f88 (diff) | |
.drop() is not a `nop` anymore
`.drop()` is now baillout out, instead of nop'ing the call.
This closes #13
Diffstat (limited to 'src/sp_disabled_functions.c')
| -rw-r--r-- | src/sp_disabled_functions.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/sp_disabled_functions.c b/src/sp_disabled_functions.c index 0f5f859..c8c723a 100644 --- a/src/sp_disabled_functions.c +++ b/src/sp_disabled_functions.c | |||
| @@ -358,7 +358,7 @@ ZEND_FUNCTION(check_disabled_function) { | |||
| 358 | const char* current_function_name = get_active_function_name(TSRMLS_C); | 358 | const char* current_function_name = get_active_function_name(TSRMLS_C); |
| 359 | 359 | ||
| 360 | if (true == should_disable(execute_data)) { | 360 | if (true == should_disable(execute_data)) { |
| 361 | return; | 361 | sp_terminate(); |
| 362 | } | 362 | } |
| 363 | 363 | ||
| 364 | if ((orig_handler = zend_hash_str_find_ptr( | 364 | if ((orig_handler = zend_hash_str_find_ptr( |
| @@ -366,7 +366,7 @@ ZEND_FUNCTION(check_disabled_function) { | |||
| 366 | strlen(current_function_name)))) { | 366 | strlen(current_function_name)))) { |
| 367 | orig_handler(INTERNAL_FUNCTION_PARAM_PASSTHRU); | 367 | orig_handler(INTERNAL_FUNCTION_PARAM_PASSTHRU); |
| 368 | if (true == should_drop_on_ret(return_value, execute_data)) { | 368 | if (true == should_drop_on_ret(return_value, execute_data)) { |
| 369 | zend_bailout(); | 369 | sp_terminate(); |
| 370 | } | 370 | } |
| 371 | } else { | 371 | } else { |
| 372 | sp_log_err( | 372 | sp_log_err( |
