Improve a bit the coverage wrt. broken configurations

3 files changed, 12 insertions, 2 deletions

diff --git a/src/sp_config_utils.c b/src/sp_config_utils.c
index 62fb7c1..12df098 100644
--- a/src/sp_config_utils.c
+++ b/src/sp_config_utils.c
@@ -60,7 +60,7 @@ static char *get_string(size_t *consumed, char *restrict line,
   size_t j = 0;
 
   char *ret = NULL;
-  if (NULL == line) {
+  if (NULL == line || '\0' == *line) {
     goto err;
   }
 
@@ -189,4 +189,4 @@ zend_always_inline sp_node_t *parse_functions_list(char *value) {
   free(tmp);
 
   return list;
-}
\ No newline at end of file
+}
diff --git a/src/tests/broken_conf_truncated.phpt b/src/tests/broken_conf_truncated.phpt
new file mode 100644
index 0000000..74f3524
--- /dev/null
+++ b/src/tests/broken_conf_truncated.phpt
@@ -0,0 +1,9 @@
+--TEST--
+Bad boolean value in configuration
+--SKIPIF--
+<?php if (!extension_loaded("snuffleupagus")) print "skip"; ?>
+--INI--
+sp.configuration_file={PWD}/config/config_broken_conf_truncated.ini
+--FILE--
+--EXPECT--
+[snuffleupagus][0.0.0.0][error][error] A valid string as parameter is expected on line 1.
diff --git a/src/tests/config/config_broken_conf_truncated.ini b/src/tests/config/config_broken_conf_truncated.ini
new file mode 100644
index 0000000..bf05dfb
--- /dev/null
+++ b/src/tests/config/config_broken_conf_truncated.ini
@@ -0,0 +1 @@
+sp.disable_function.function("").param(no quote, omg!