4 files changed, 35 insertions, 6 deletions

diff --git a/doc/source/config.rst b/doc/source/config.rst
index dd30723..258b1ab 100644
--- a/doc/source/config.rst
+++ b/doc/source/config.rst
@@ -328,7 +328,8 @@ The ``function`` filter is able to do various dereferencing:
 - ``function("AwesomeNamespace\\my_function")`` will match the function ``my_function`` in the namespace ``AwesomeNamespace``
 
 It's also able to have calltrace constrains: ``function(func1>func2)`` will
-match only if ``func2`` is called **inside** of ``func1``.
+match only if ``func2`` is called **inside** of ``func1``. Do note that their
+might be other functions called between them.
 
 The ``param`` filter is able to do some dereferencing as well:
 
diff --git a/src/sp_disabled_functions.c b/src/sp_disabled_functions.c
index 7be1c34..7e6ca6a 100644
--- a/src/sp_disabled_functions.c
+++ b/src/sp_disabled_functions.c
@@ -40,7 +40,7 @@ static bool is_functions_list_matching(zend_execute_data* execute_data,
                                        sp_list_node* functions_list) {
   zend_execute_data *orig_execute_data, *current;
   orig_execute_data = current = execute_data;
-  sp_list_node* it = functions_list;
+  sp_list_node const * it = functions_list;
 
   while (current) {
     if (it == NULL) {  // every function in the list matched, we've got a match!
@@ -50,7 +50,7 @@ static bool is_functions_list_matching(zend_execute_data* execute_data,
 
     EG(current_execute_data) = current;
 
-    char* complete_path_function = get_complete_function_path(current);
+    char* const complete_path_function = get_complete_function_path(current);
     if (!complete_path_function) {
       break;
     }
@@ -59,10 +59,8 @@ static bool is_functions_list_matching(zend_execute_data* execute_data,
 
     if (0 == match) {
       it = it->next;
-      current = current->prev_execute_data;
-    } else {
-      break;
     }
+                current = current->prev_execute_data;
   }
 
   EG(current_execute_data) = orig_execute_data;
diff --git a/src/tests/disable_function/config/config_disabled_functions_chain_call_skip.ini b/src/tests/disable_function/config/config_disabled_functions_chain_call_skip.ini
new file mode 100644
index 0000000..4d2f68d
--- /dev/null
+++ b/src/tests/disable_function/config/config_disabled_functions_chain_call_skip.ini
@@ -0,0 +1 @@
+sp.disable_function.function("a>c").simulation().drop();
diff --git a/src/tests/disable_function/disabled_functions_chain_call_skip.phpt b/src/tests/disable_function/disabled_functions_chain_call_skip.phpt
new file mode 100644
index 0000000..9ff84b9
--- /dev/null
+++ b/src/tests/disable_function/disabled_functions_chain_call_skip.phpt
@@ -0,0 +1,29 @@
+--TEST--
+Disable functions by matching the calltrace, with a superfluous function in between
+--SKIPIF--
+<?php if (!extension_loaded("snuffleupagus")) die "skip"; ?>
+--INI--
+sp.configuration_file={PWD}/config/config_disabled_functions_chain_call_skip.ini
+--FILE--
+<?php 
+
+function a() {
+        echo "I'm in the `a` function!\n";
+        b();
+}
+function b() {
+        echo "I'm in the `b` function!\n";
+        c();
+}
+function c() {
+        echo "I'm in the `c` function!\n";
+}
+
+a();
+?>
+--EXPECTF--
+I'm in the `a` function!
+I'm in the `b` function!
+
+Warning: [snuffleupagus][0.0.0.0][disabled_function][simulation] Aborted execution on call of the function 'a>c' in %s/tests/disable_function/disabled_functions_chain_call_skip.php on line 12
+I'm in the `c` function!