From ef1cb7661ed577039c7fa6caea7490330237f091 Mon Sep 17 00:00:00 2001
From: jvoisin
Date: Tue, 10 Oct 2017 14:47:56 +0200
Subject: Bump coverage, and fix a segfault on trace matching

---
 src/sp_config_keywords.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

(limited to 'src/sp_config_keywords.c')

diff --git a/src/sp_config_keywords.c b/src/sp_config_keywords.c
index 8fba868..e71463a 100644
--- a/src/sp_config_keywords.c
+++ b/src/sp_config_keywords.c
@@ -128,13 +128,16 @@ int parse_cookie_encryption(char *line) {
       "on line %zu without having set the `.encryption_key` option in"
       "`sp.global`: please set it first.", sp_line_no);
     return -1;
+  } else if (0 == strlen(name)) {
+    sp_log_err("config", "You must specify a cookie name to encrypt on line "
+      "%zu.", sp_line_no);
+    return -1;
   }
 
-  if (name) {
-    zend_hash_str_add_empty_element(
-        SNUFFLEUPAGUS_G(config).config_cookie_encryption->names, name,
-        strlen(name));
-  }
+  zend_hash_str_add_empty_element(
+      SNUFFLEUPAGUS_G(config).config_cookie_encryption->names, name,
+      strlen(name));
+
   return SUCCESS;
 }
 
-- 
cgit v1.3