Bump a bit the coverage

3 files changed, 11 insertions, 2 deletions

diff --git a/src/sp_disabled_functions.c b/src/sp_disabled_functions.c
index 5619e1f..f5051df 100644
--- a/src/sp_disabled_functions.c
+++ b/src/sp_disabled_functions.c
@@ -193,8 +193,7 @@ static bool is_param_matching(zend_execute_data* execute_data,
     if (arg_value) {
       *arg_value_str = sp_convert_to_string(arg_value);
       if (config_node->param_type) {  // Are we matching on the `type`?
-        if (config_node->param_type &&
-            config_node->param_type == Z_TYPE_P(arg_value)) {
+        if (config_node->param_type == Z_TYPE_P(arg_value)) {
           return true;
         }
       } else if (Z_TYPE_P(arg_value) == IS_ARRAY) {
diff --git a/src/tests/broken_conf_eval.phpt b/src/tests/broken_conf_eval.phpt
new file mode 100644
index 0000000..4dc5312
--- /dev/null
+++ b/src/tests/broken_conf_eval.phpt
@@ -0,0 +1,9 @@
+--TEST--
+Broken configuration for eval
+--SKIPIF--
+<?php if (!extension_loaded("snuffleupagus")) print "skip"; ?>
+--INI--
+sp.configuration_file={PWD}/config/broken_conf_eval.ini
+--FILE--
+--EXPECT--
+[snuffleupagus][0.0.0.0][error][error] There is an issue with the parsing of '"cos,sin': it doesn't look like a valid string on line 1.
diff --git a/src/tests/config/broken_conf_eval.ini b/src/tests/config/broken_conf_eval.ini
new file mode 100644
index 0000000..80ef7e5
--- /dev/null
+++ b/src/tests/config/broken_conf_eval.ini
@@ -0,0 +1 @@
+sp.eval_blacklist.list("cos,sin