diff options
| author | jvoisin | 2017-12-01 17:28:44 +0100 |
|---|---|---|
| committer | jvoisin | 2017-12-01 17:28:44 +0100 |
| commit | 86dac0e21cb9547ce1544539bb0f2c2f73ca5f73 (patch) | |
| tree | 6be4523305a6095d9091795b62ba4853dcf4a4a2 /src | |
| parent | 5efe641fefc74dfc908efd43d862477e91d904a9 (diff) | |
Fail sooner when not able to create the folder to dump
Diffstat (limited to 'src')
| -rw-r--r-- | src/sp_utils.c | 1 | ||||
| -rw-r--r-- | src/tests/dump_request_invalid_folder.phpt | 1 |
2 files changed, 1 insertions, 1 deletions
diff --git a/src/sp_utils.c b/src/sp_utils.c index 73ad528..d4ca73d 100644 --- a/src/sp_utils.c +++ b/src/sp_utils.c | |||
| @@ -88,6 +88,7 @@ static int construct_filename(char* filename, const char* folder) { | |||
| 88 | if (0 > mkdir(folder, 0700) && errno != EEXIST) { | 88 | if (0 > mkdir(folder, 0700) && errno != EEXIST) { |
| 89 | sp_log_err("request_logging", "Unable to create the folder '%s'.", | 89 | sp_log_err("request_logging", "Unable to create the folder '%s'.", |
| 90 | folder); | 90 | folder); |
| 91 | return -1; | ||
| 91 | } | 92 | } |
| 92 | 93 | ||
| 93 | memcpy(filename, folder, strlen(folder)); | 94 | memcpy(filename, folder, strlen(folder)); |
diff --git a/src/tests/dump_request_invalid_folder.phpt b/src/tests/dump_request_invalid_folder.phpt index f8e3b43..6d395d0 100644 --- a/src/tests/dump_request_invalid_folder.phpt +++ b/src/tests/dump_request_invalid_folder.phpt | |||
| @@ -22,4 +22,3 @@ echo "2\n"; | |||
| 22 | 1 | 22 | 1 |
| 23 | [snuffleupagus][0.0.0.0][disabled_function][drop] The call to the function 'system' in %atests/dump_request_invalid_folder.php:3 has been disabled. | 23 | [snuffleupagus][0.0.0.0][disabled_function][drop] The call to the function 'system' in %atests/dump_request_invalid_folder.php:3 has been disabled. |
| 24 | [snuffleupagus][0.0.0.0][request_logging][error] Unable to create the folder '/root/NON_EXISTENT/FOLDER/PLEASE/'. | 24 | [snuffleupagus][0.0.0.0][request_logging][error] Unable to create the folder '/root/NON_EXISTENT/FOLDER/PLEASE/'. |
| 25 | [snuffleupagus][0.0.0.0][request_logging][error] Unable to open /root/NON_EXISTENT/FOLDER/PLEASE/sp_dump_%a_0.0.0.0.dump | ||
