From 424845a6938ffa9fba850b2285e2ab4ce60c89d0 Mon Sep 17 00:00:00 2001
From: jvoisin
Date: Fri, 1 Dec 2017 17:59:07 +0100
Subject: Add a test that used to segfault

---
 src/sp_utils.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

(limited to 'src/sp_utils.c')

diff --git a/src/sp_utils.c b/src/sp_utils.c
index d4ca73d..e2747fb 100644
--- a/src/sp_utils.c
+++ b/src/sp_utils.c
@@ -118,13 +118,16 @@ int sp_log_request(const char* folder) {
     const char* str;
     const int key;
   } zones[] = {{"GET", TRACK_VARS_GET},       {"POST", TRACK_VARS_POST},
-               {"COOKIE", TRACK_VARS_COOKIE}, {"SERVER", TRACK_VARS_SERVER},
-               {"ENV", TRACK_VARS_ENV},       {NULL, 0}};
+               {"COOKIE", TRACK_VARS_COOKIE}, /*{"SERVER", TRACK_VARS_SERVER}, */
+               {"ENV", TRACK_VARS_ENV},			  /*{"REQUEST", TRACK_VARS_REQUEST},*/
+							 {NULL, 0}};
+	// Apparently, PHP has trouble always giving SERVER,
+	// and REQUEST is never used in its source code.
 
   if (0 != construct_filename(filename, folder)) {
     return -1;
   }
-  if (NULL == (file = fopen(filename, "a"))) {
+  if (NULL == (file = fopen(filename, "w+"))) {
     sp_log_err("request_logging", "Unable to open %s", filename);
     return -1;
   }
-- 
cgit v1.3