diff options
| author | jvoisin | 2017-12-01 17:59:07 +0100 |
|---|---|---|
| committer | jvoisin | 2017-12-01 17:59:07 +0100 |
| commit | 424845a6938ffa9fba850b2285e2ab4ce60c89d0 (patch) | |
| tree | fac7acba8385fe16e662ce37754aed57d646a526 /src/sp_utils.c | |
| parent | 86dac0e21cb9547ce1544539bb0f2c2f73ca5f73 (diff) | |
Add a test that used to segfault
Diffstat (limited to 'src/sp_utils.c')
| -rw-r--r-- | src/sp_utils.c | 9 |
1 files changed, 6 insertions, 3 deletions
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) { | |||
| 118 | const char* str; | 118 | const char* str; |
| 119 | const int key; | 119 | const int key; |
| 120 | } zones[] = {{"GET", TRACK_VARS_GET}, {"POST", TRACK_VARS_POST}, | 120 | } zones[] = {{"GET", TRACK_VARS_GET}, {"POST", TRACK_VARS_POST}, |
| 121 | {"COOKIE", TRACK_VARS_COOKIE}, {"SERVER", TRACK_VARS_SERVER}, | 121 | {"COOKIE", TRACK_VARS_COOKIE}, /*{"SERVER", TRACK_VARS_SERVER}, */ |
| 122 | {"ENV", TRACK_VARS_ENV}, {NULL, 0}}; | 122 | {"ENV", TRACK_VARS_ENV}, /*{"REQUEST", TRACK_VARS_REQUEST},*/ |
| 123 | {NULL, 0}}; | ||
| 124 | // Apparently, PHP has trouble always giving SERVER, | ||
| 125 | // and REQUEST is never used in its source code. | ||
| 123 | 126 | ||
| 124 | if (0 != construct_filename(filename, folder)) { | 127 | if (0 != construct_filename(filename, folder)) { |
| 125 | return -1; | 128 | return -1; |
| 126 | } | 129 | } |
| 127 | if (NULL == (file = fopen(filename, "a"))) { | 130 | if (NULL == (file = fopen(filename, "w+"))) { |
| 128 | sp_log_err("request_logging", "Unable to open %s", filename); | 131 | sp_log_err("request_logging", "Unable to open %s", filename); |
| 129 | return -1; | 132 | return -1; |
| 130 | } | 133 | } |
