From: Viktor Szakats <commit@vsz.me>
Date: Sun, 6 Oct 2024 13:41:48 +0000 (+0200)
Subject: build: detect and use `_setmode()` with Cygwin/MSYS, also use on Windows
X-Git-Tag: curl-8_11_0~222
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=5e70566094463b8463d4dc1bec9b66763a761c76;p=thirdparty%2Fcurl.git

build: detect and use `_setmode()` with Cygwin/MSYS, also use on Windows

Before this patch `setmode()` was not detected with Cygwin/MSYS, because
it's a macro, not a function, and detection is looking for a function.

Switching to symbol detection doesn't work because it mis-detects it on
BSD systems which features a function with the same name but different
functionality and arguments.

Fix it by looking for a `_setmode()` function on Cygwin/MSYS, and use it
if available.

`_setmode()` is recommended over `setmode()` by Windows documentation so
use that on Windows too. It seems to be available on all supported
compilers, so omit detection.

https://learn.microsoft.com/cpp/c-runtime-library/reference/posix-setmode
https://learn.microsoft.com/cpp/c-runtime-library/reference/setmode

Officially Windows requires argument `_O_BINARY` with an underscore.
`O_BINARY` is also supported but bound to conditions. Continue to use it
for simplicity. Cygwin supports `O_BINARY` (no underscore).

Closes #15169
---

diff --git a/CMake/Platforms/WindowsCache.cmake b/CMake/Platforms/WindowsCache.cmake
index 1cedad3fd7..668bba215a 100644
--- a/CMake/Platforms/WindowsCache.cmake
+++ b/CMake/Platforms/WindowsCache.cmake
@@ -105,6 +105,7 @@ set(HAVE_FSETXATTR 0)
 set(HAVE_LIBSOCKET 0)
 set(HAVE_SETLOCALE 1)
 set(HAVE_SETMODE 1)
+set(HAVE__SETMODE 1)
 set(HAVE_GETPEERNAME 1)
 set(HAVE_GETSOCKNAME 1)
 set(HAVE_GETHOSTNAME 1)
diff --git a/CMakeLists.txt b/CMakeLists.txt
index f75c1ee7d7..0e2ae7a1b9 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -1524,6 +1524,10 @@ check_symbol_exists("setlocale"       "${CURL_INCLUDES}" HAVE_SETLOCALE)
 check_symbol_exists("setmode"         "${CURL_INCLUDES}" HAVE_SETMODE)
 check_symbol_exists("setrlimit"       "${CURL_INCLUDES}" HAVE_SETRLIMIT)
 
+if(WIN32 OR CYGWIN)
+  check_function_exists("_setmode" HAVE__SETMODE)
+endif()
+
 if(NOT _ssl_enabled)
   check_symbol_exists("arc4random" "${CURL_INCLUDES};stdlib.h" HAVE_ARC4RANDOM)
 endif()
diff --git a/acinclude.m4 b/acinclude.m4
index df893e05b8..9afc1774da 100644
--- a/acinclude.m4
+++ b/acinclude.m4
@@ -1553,9 +1553,9 @@ AC_DEFUN([CURL_PREPARE_BUILDINFO], [
         curl_pflags="${curl_pflags} UNIX";;
     esac
   fi
-  case $host_os in
-    cygwin*|msys*) curl_pflags="${curl_pflags} CYGWIN";;
-  esac
+  if test "$curl_cv_cygwin" = 'yes'; then
+    curl_pflags="${curl_pflags} CYGWIN"
+  fi
   case $host_os in
     msys*) curl_pflags="${curl_pflags} MSYS";;
   esac
diff --git a/configure.ac b/configure.ac
index e5998a1062..9bf2f51b3f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -605,6 +605,11 @@ case $host in
     ;;
 esac
 
+curl_cv_cygwin='no'
+case $host_os in
+  cygwin*|msys*) curl_cv_cygwin='yes';;
+esac
+
 AM_CONDITIONAL([HAVE_WINDRES],
   [test "$curl_cv_native_windows" = "yes" && test -n "${RC}"])
 
@@ -4120,6 +4125,10 @@ AC_CHECK_FUNCS([\
   utimes \
 ])
 
+if test "$curl_cv_native_windows" = 'yes' -o "$curl_cv_cygwin" = 'yes'; then
+  AC_CHECK_FUNCS([_setmode])
+fi
+
 if test -z "$ssl_backends"; then
   AC_CHECK_FUNCS([arc4random])
 fi
diff --git a/lib/config-win32.h b/lib/config-win32.h
index b855e5c635..c8fdf6df09 100644
--- a/lib/config-win32.h
+++ b/lib/config-win32.h
@@ -155,6 +155,9 @@
 /* Define if you have the setmode function. */
 #define HAVE_SETMODE 1
 
+/* Define if you have the _setmode function. */
+#define HAVE__SETMODE 1
+
 /* Define if you have the socket function. */
 #define HAVE_SOCKET 1
 
diff --git a/lib/curl_config.h.cmake b/lib/curl_config.h.cmake
index dd806da9bc..e69fc50073 100644
--- a/lib/curl_config.h.cmake
+++ b/lib/curl_config.h.cmake
@@ -481,6 +481,9 @@
 /* Define to 1 if you have the `setmode' function. */
 #cmakedefine HAVE_SETMODE 1
 
+/* Define to 1 if you have the `_setmode' function. */
+#cmakedefine HAVE__SETMODE 1
+
 /* Define to 1 if you have the `setrlimit' function. */
 #cmakedefine HAVE_SETRLIMIT 1
 
diff --git a/src/tool_binmode.c b/src/tool_binmode.c
index e27ce9663c..a92596416e 100644
--- a/src/tool_binmode.c
+++ b/src/tool_binmode.c
@@ -23,7 +23,7 @@
  ***************************************************************************/
 #include "tool_setup.h"
 
-#ifdef HAVE_SETMODE
+#if defined(HAVE_SETMODE) || defined(HAVE__SETMODE)
 
 #ifdef HAVE_IO_H
 #  include <io.h>
@@ -42,6 +42,8 @@ void set_binmode(FILE *stream)
 #ifdef O_BINARY
 #  ifdef __HIGHC__
   _setmode(stream, O_BINARY);
+#  elif defined(HAVE__SETMODE)
+  (void)_setmode(fileno(stream), O_BINARY);
 #  else
   (void)setmode(fileno(stream), O_BINARY);
 #  endif
@@ -50,4 +52,4 @@ void set_binmode(FILE *stream)
 #endif
 }
 
-#endif /* HAVE_SETMODE */
+#endif /* HAVE_SETMODE || HAVE__SETMODE */
diff --git a/src/tool_binmode.h b/src/tool_binmode.h
index bee837b000..ce2f589e0f 100644
--- a/src/tool_binmode.h
+++ b/src/tool_binmode.h
@@ -25,7 +25,7 @@
  ***************************************************************************/
 #include "tool_setup.h"
 
-#ifdef HAVE_SETMODE
+#if defined(HAVE_SETMODE) || defined(HAVE__SETMODE)
 
 void set_binmode(FILE *stream);
 
@@ -33,6 +33,6 @@ void set_binmode(FILE *stream);
 
 #define set_binmode(x) Curl_nop_stmt
 
-#endif /* HAVE_SETMODE */
+#endif /* HAVE_SETMODE || HAVE__SETMODE */
 
 #endif /* HEADER_CURL_TOOL_BINMODE_H */