Fall back to unsigned int, not int, for socklen_t.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 11 Nov 2021 15:36:39 +0000 (10:36 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 11 Nov 2021 15:36:39 +0000 (10:36 -0500)
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 ee3a1a5b6.

Discussion: https://postgr.es/m/1440792.1636558888@sss.pgh.pa.us

src/include/port.h

index ae68d9c3c7938c9128fc0675d721c64f8a1f839f..49b4d38131bf84a2bf5173a41ff942e9630be2be 100644 (file)
@@ -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 */