Fall back to unsigned int, not int, for socklen_t.

tglsfdc · tglsfdc · commit 01ec41a5fe4a · 2021-11-11T10:36:39.000-05:00
It's a coin toss which of these is a better default assumption. However, of the machines we have in the buildfarm, the only ones relying on the fallback socklen_t definition are ancient HPUX, and on that platform unsigned int is the right choice. Minor tweak to ee3a1a5. Discussion: https://postgr.es/m/1440792.1636558888@sss.pgh.pa.us
diff --git a/src/include/port.h b/src/include/port.h
@@ -37,8 +37,9 @@ typedef SOCKET pgsocket;
 #define PGINVALID_SOCKET INVALID_SOCKET
 #endif
 
+/* if platform lacks socklen_t, we assume this will work */
 #ifndef HAVE_SOCKLEN_T
-typedef int socklen_t;
+typedef unsigned int socklen_t;
 #endif
 
 /* non-blocking */
