Add a test for unmatched brackets

3 files changed, 12 insertions, 3 deletions

diff --git a/src/sp_config_utils.c b/src/sp_config_utils.c
index f562ffd..62fb7c1 100644
--- a/src/sp_config_utils.c
+++ b/src/sp_config_utils.c
@@ -2,9 +2,7 @@
 
 size_t sp_line_no;
 
-static int validate_str(const char *value);
-
-static sp_pure int validate_str(const char *value) {
+static int validate_str(const char *value) {
   int balance = 0;  // ghetto [] validation
 
   if (!strchr(value, '[')) {
@@ -18,6 +16,7 @@ static sp_pure int validate_str(const char *value) {
       balance--;
     }
     if (balance < 0) {
+      sp_log_err("config", "The string '%s' contains unbalanced brackets.", value);
       return -1;
     }
   }
diff --git a/src/tests/broken_unmatching_brackets.phpt b/src/tests/broken_unmatching_brackets.phpt
new file mode 100644
index 0000000..14b9414
--- /dev/null
+++ b/src/tests/broken_unmatching_brackets.phpt
@@ -0,0 +1,9 @@
+--TEST--
+Broken configuration - unmatching brackets
+--SKIPIF--
+<?php if (!extension_loaded("snuffleupagus")) die "skip"; ?>
+--INI--
+sp.configuration_file={PWD}/config/config_unmatching_brackets.ini
+--FILE--
+--EXPECTF--
+[snuffleupagus][0.0.0.0][config][error] The string 'arr[b]]]]]' contains unbalanced brackets.
diff --git a/src/tests/config/config_unmatching_brackets.ini b/src/tests/config/config_unmatching_brackets.ini
new file mode 100644
index 0000000..45fa4fe
--- /dev/null
+++ b/src/tests/config/config_unmatching_brackets.ini
@@ -0,0 +1 @@
+sp.disable_function.function("foo").param("arr[b]]]]]").value("aaa").alias("4").drop();