tool_cb_wrt: fix outfile mode flags for Windows

- Use S_IREAD and S_IWRITE mode permission flags to create the file
  on Windows instead of S_IRUSR, S_IWUSR, etc.

Windows only accepts a combination of S_IREAD and S_IWRITE. It does not
acknowledge other combinations, for which it may generate an assertion.

This is a follow-up to 81b4e99 from yesterday, which improved the
existing file check with -J.

Ref: https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/open-wopen#remarks
Ref: https://github.com/curl/curl/pull/5731

Closes https://github.com/curl/curl/pull/5742

diff --git a/src/tool_cb_wrt.c b/src/tool_cb_wrt.c
index e0742630ba658a23c605087e8a73ff3cbc1167fc..64b62fefd4a499e1d933e6c665f17816eef1b388 100644 (file)
--- a/src/tool_cb_wrt.c
+++ b/src/tool_cb_wrt.c
@@ -26,6 +26,8 @@
 #include <fcntl.h>
 #endif
 
+#include <sys/stat.h>
+
 #define ENABLE_CURLX_PRINTF
 /* use our own printf() functions */
 #include "curlx.h"
@@ -57,9 +59,10 @@ bool tool_create_output_file(struct OutStruct *outs,
 #define O_BINARY 0
 #endif
     int fd = open(outs->filename, O_CREAT | O_WRONLY | O_EXCL | O_BINARY,
-                  S_IRUSR | S_IWUSR
-#ifdef S_IRGRP
-                  | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH
+#ifdef WIN32
+                  S_IREAD | S_IWRITE
+#else
+                  S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH
 #endif
       );
     if(fd != -1) {