From 8f94a1179e9cf9d96f18ec6c851dddbecd54ea6f Mon Sep 17 00:00:00 2001
From: Thibault "bui" Koechlin
Date: Wed, 18 Oct 2017 13:17:46 +0200
Subject: .drop() is not a `nop` anymore

`.drop()` is now baillout out, instead of nop'ing the call.

This closes #13---
 src/sp_disabled_functions.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src/sp_disabled_functions.c')

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) {
   const char* current_function_name = get_active_function_name(TSRMLS_C);
 
   if (true == should_disable(execute_data)) {
-    return;
+    sp_terminate();
   }
 
   if ((orig_handler = zend_hash_str_find_ptr(
@@ -366,7 +366,7 @@ ZEND_FUNCTION(check_disabled_function) {
            strlen(current_function_name)))) {
     orig_handler(INTERNAL_FUNCTION_PARAM_PASSTHRU);
     if (true == should_drop_on_ret(return_value, execute_data)) {
-      zend_bailout();
+      sp_terminate();
     }
   } else {
     sp_log_err(
-- 
cgit v1.3