Remove check for accept() argument types
authorPeter Eisentraut <peter@eisentraut.org>
Tue, 9 Nov 2021 14:20:47 +0000 (15:20 +0100)
committerPeter Eisentraut <peter@eisentraut.org>
Tue, 9 Nov 2021 14:35:26 +0000 (15:35 +0100)
This check was used to accommodate a staggering variety in particular
in the type of the third argument of accept().  This is no longer of
concern on currently supported systems.  We can just use socklen_t in
the code and put in a simple check that substitutes int for socklen_t
if it's missing, to cover the few stragglers.

Reviewed-by: Andres Freund <andres@anarazel.de>
Discussion: https://www.postgresql.org/message-id/3538f4c4-1886-64f2-dcff-aaad8267fb82@enterprisedb.com

13 files changed:
aclocal.m4
config/ac_func_accept_argtypes.m4 [deleted file]
configure
configure.ac
src/backend/libpq/auth.c
src/backend/libpq/pqcomm.c
src/backend/postmaster/pgstat.c
src/include/libpq/pqcomm.h
src/include/pg_config.h.in
src/include/port.h
src/interfaces/libpq/fe-connect.c
src/port/getpeereid.c
src/tools/msvc/Solution.pm

index 5e22482cd53de3330b0983465b96305b8404b6e5..58ade65046d0a5adfe1ae5416637c76ae7242cd7 100644 (file)
@@ -1,5 +1,4 @@
 dnl aclocal.m4
-m4_include([config/ac_func_accept_argtypes.m4])
 m4_include([config/ax_prog_perl_modules.m4])
 m4_include([config/ax_pthread.m4])
 m4_include([config/c-compiler.m4])
diff --git a/config/ac_func_accept_argtypes.m4 b/config/ac_func_accept_argtypes.m4
deleted file mode 100644 (file)
index 178ef67..0000000
+++ /dev/null
@@ -1,78 +0,0 @@
-# config/ac_func_accept_argtypes.m4
-# This comes from the official Autoconf macro archive at
-# <http://research.cys.de/autoconf-archive/>
-
-
-dnl @synopsis AC_FUNC_ACCEPT_ARGTYPES
-dnl
-dnl Checks the data types of the three arguments to accept(). Results are
-dnl placed into the symbols ACCEPT_TYPE_RETURN and ACCEPT_TYPE_ARG[123],
-dnl consistent with the following example:
-dnl
-dnl       #define ACCEPT_TYPE_RETURN int
-dnl       #define ACCEPT_TYPE_ARG1 int
-dnl       #define ACCEPT_TYPE_ARG2 struct sockaddr *
-dnl       #define ACCEPT_TYPE_ARG3 socklen_t
-dnl
-dnl NOTE: This is just a modified version of the AC_FUNC_SELECT_ARGTYPES
-dnl macro. Credit for that one goes to David MacKenzie et. al.
-dnl
-dnl @version $Id: ac_func_accept_argtypes.m4,v 1.1 1999/12/03 11:29:29 simons Exp $
-dnl @author Daniel Richard G. <skunk@mit.edu>
-dnl
-
-# PostgreSQL local changes: In the original version ACCEPT_TYPE_ARG3
-# is a pointer type. That's kind of useless because then you can't
-# use the macro to define a corresponding variable. We also make the
-# reasonable(?) assumption that you can use arg3 for getsocktype etc.
-# as well (i.e., anywhere POSIX.2 has socklen_t).
-#
-# arg2 can also be `const' (e.g., RH 4.2). Change the order of tests
-# for arg3 so that `int' is first, in case there is no prototype at all.
-#
-# Solaris 7 and 8 have arg3 as 'void *' (disguised as 'Psocklen_t'
-# which is *not* 'socklen_t *').  If we detect that, then we assume
-# 'int' as the result, because that ought to work best.
-#
-# On Win32, accept() returns 'unsigned int PASCAL'
-# Win64 uses SOCKET for return and arg1
-
-AC_DEFUN([AC_FUNC_ACCEPT_ARGTYPES],
-[AC_MSG_CHECKING([types of arguments for accept()])
- AC_CACHE_VAL(ac_cv_func_accept_return,dnl
- [AC_CACHE_VAL(ac_cv_func_accept_arg1,dnl
-  [AC_CACHE_VAL(ac_cv_func_accept_arg2,dnl
-   [AC_CACHE_VAL(ac_cv_func_accept_arg3,dnl
-    [for ac_cv_func_accept_return in 'int' 'SOCKET WSAAPI' 'unsigned int PASCAL'; do
-      for ac_cv_func_accept_arg1 in 'int' 'SOCKET' 'unsigned int'; do
-       for ac_cv_func_accept_arg2 in 'struct sockaddr *' 'const struct sockaddr *' 'void *'; do
-        for ac_cv_func_accept_arg3 in 'int' 'size_t' 'socklen_t' 'unsigned int' 'void'; do
-         AC_COMPILE_IFELSE([AC_LANG_SOURCE(
-[#include <sys/types.h>
-#include <sys/socket.h>
-extern $ac_cv_func_accept_return accept ($ac_cv_func_accept_arg1, $ac_cv_func_accept_arg2, $ac_cv_func_accept_arg3 *);])],
-         [ac_not_found=no; break 4], [ac_not_found=yes])
-       done
-      done
-     done
-    done
-    if test "$ac_not_found" = yes; then
-      AC_MSG_ERROR([could not determine argument types])
-    fi
-    if test "$ac_cv_func_accept_arg3" = "void"; then
-      ac_cv_func_accept_arg3=int
-    fi
-    ])dnl AC_CACHE_VAL
-   ])dnl AC_CACHE_VAL
-  ])dnl AC_CACHE_VAL
- ])dnl AC_CACHE_VAL
- AC_MSG_RESULT([$ac_cv_func_accept_return, $ac_cv_func_accept_arg1, $ac_cv_func_accept_arg2, $ac_cv_func_accept_arg3 *])
- AC_DEFINE_UNQUOTED(ACCEPT_TYPE_RETURN, $ac_cv_func_accept_return,
-                    [Define to the return type of 'accept'])
- AC_DEFINE_UNQUOTED(ACCEPT_TYPE_ARG1, $ac_cv_func_accept_arg1,
-                    [Define to the type of arg 1 of 'accept'])
- AC_DEFINE_UNQUOTED(ACCEPT_TYPE_ARG2, $ac_cv_func_accept_arg2,
-                    [Define to the type of arg 2 of 'accept'])
- AC_DEFINE_UNQUOTED(ACCEPT_TYPE_ARG3, $ac_cv_func_accept_arg3,
-                    [Define to the type of arg 3 of 'accept'])
-])
index 4ffefe46552de0cd992e0ccd84066397fc3d2264..e60e78efdf2a47bb3189ccfab64369216fadf071 100755 (executable)
--- a/configure
+++ b/configure
@@ -14615,6 +14615,17 @@ cat >>confdefs.h <<_ACEOF
 _ACEOF
 
 
+fi
+
+ac_fn_c_check_type "$LINENO" "socklen_t" "ac_cv_type_socklen_t" "#include <sys/socket.h>
+"
+if test "x$ac_cv_type_socklen_t" = xyes; then :
+
+cat >>confdefs.h <<_ACEOF
+#define HAVE_SOCKLEN_T 1
+_ACEOF
+
+
 fi
 
 ac_fn_c_check_type "$LINENO" "struct sockaddr_un" "ac_cv_type_struct_sockaddr_un" "#include <sys/types.h>
@@ -15327,77 +15338,6 @@ if test x"$pgac_cv_var_int_timezone" = xyes ; then
 $as_echo "#define HAVE_INT_TIMEZONE 1" >>confdefs.h
 
 fi
-{ $as_echo "$as_me:${as_lineno-$LINENO}: checking types of arguments for accept()" >&5
-$as_echo_n "checking types of arguments for accept()... " >&6; }
- if ${ac_cv_func_accept_return+:} false; then :
-  $as_echo_n "(cached) " >&6
-else
-   if ${ac_cv_func_accept_arg1+:} false; then :
-  $as_echo_n "(cached) " >&6
-else
-    if ${ac_cv_func_accept_arg2+:} false; then :
-  $as_echo_n "(cached) " >&6
-else
-     if ${ac_cv_func_accept_arg3+:} false; then :
-  $as_echo_n "(cached) " >&6
-else
-      for ac_cv_func_accept_return in 'int' 'SOCKET WSAAPI' 'unsigned int PASCAL'; do
-      for ac_cv_func_accept_arg1 in 'int' 'SOCKET' 'unsigned int'; do
-       for ac_cv_func_accept_arg2 in 'struct sockaddr *' 'const struct sockaddr *' 'void *'; do
-        for ac_cv_func_accept_arg3 in 'int' 'size_t' 'socklen_t' 'unsigned int' 'void'; do
-         cat confdefs.h - <<_ACEOF >conftest.$ac_ext
-/* end confdefs.h.  */
-#include <sys/types.h>
-#include <sys/socket.h>
-extern $ac_cv_func_accept_return accept ($ac_cv_func_accept_arg1, $ac_cv_func_accept_arg2, $ac_cv_func_accept_arg3 *);
-_ACEOF
-if ac_fn_c_try_compile "$LINENO"; then :
-  ac_not_found=no; break 4
-else
-  ac_not_found=yes
-fi
-rm -f core conftest.err conftest.$ac_objext conftest.$ac_ext
-       done
-      done
-     done
-    done
-    if test "$ac_not_found" = yes; then
-      as_fn_error $? "could not determine argument types" "$LINENO" 5
-    fi
-    if test "$ac_cv_func_accept_arg3" = "void"; then
-      ac_cv_func_accept_arg3=int
-    fi
-
-fi
-
-fi
-
-fi
-
-fi
- { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_cv_func_accept_return, $ac_cv_func_accept_arg1, $ac_cv_func_accept_arg2, $ac_cv_func_accept_arg3 *" >&5
-$as_echo "$ac_cv_func_accept_return, $ac_cv_func_accept_arg1, $ac_cv_func_accept_arg2, $ac_cv_func_accept_arg3 *" >&6; }
-
-cat >>confdefs.h <<_ACEOF
-#define ACCEPT_TYPE_RETURN $ac_cv_func_accept_return
-_ACEOF
-
-
-cat >>confdefs.h <<_ACEOF
-#define ACCEPT_TYPE_ARG1 $ac_cv_func_accept_arg1
-_ACEOF
-
-
-cat >>confdefs.h <<_ACEOF
-#define ACCEPT_TYPE_ARG2 $ac_cv_func_accept_arg2
-_ACEOF
-
-
-cat >>confdefs.h <<_ACEOF
-#define ACCEPT_TYPE_ARG3 $ac_cv_func_accept_arg3
-_ACEOF
-
-
 { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether gettimeofday takes only one argument" >&5
 $as_echo_n "checking whether gettimeofday takes only one argument... " >&6; }
 if ${pgac_cv_func_gettimeofday_1arg+:} false; then :
index 44ee3ebe2f143e921c013a26679ccb49c2c76294..22cabe1b6059ff6c9f5a248149d8cf655193f028 100644 (file)
@@ -1552,6 +1552,7 @@ PGAC_C_BUILTIN_UNREACHABLE
 PGAC_C_COMPUTED_GOTO
 PGAC_STRUCT_TIMEZONE
 PGAC_UNION_SEMUN
+AC_CHECK_TYPES(socklen_t, [], [], [#include <sys/socket.h>])
 PGAC_STRUCT_SOCKADDR_UN
 PGAC_STRUCT_SOCKADDR_STORAGE
 PGAC_STRUCT_SOCKADDR_STORAGE_MEMBERS
@@ -1686,7 +1687,6 @@ fi
 ##
 
 PGAC_VAR_INT_TIMEZONE
-AC_FUNC_ACCEPT_ARGTYPES
 PGAC_FUNC_GETTIMEOFDAY_1ARG
 PGAC_FUNC_WCSTOMBS_L
 
index a317aef1c92337771ee93310e28a80bb11cb00a5..7bcf52523b48d9dd3d1963756f4147c366ea752a 100644 (file)
@@ -3026,7 +3026,7 @@ PerformRadiusTransaction(const char *server, const char *secret, const char *por
        struct addrinfo hint;
        struct addrinfo *serveraddrs;
        int                     port;
-       ACCEPT_TYPE_ARG3 addrsize;
+       socklen_t       addrsize;
        fd_set          fdset;
        struct timeval endtime;
        int                     i,
index 44782f2d88d30fc04b40e4f85cf1812e1531fa76..9ebba025cc7442a57387fdbcd6f2f6150bda15ca 100644 (file)
@@ -1632,7 +1632,7 @@ pq_getkeepalivesidle(Port *port)
        if (port->default_keepalives_idle == 0)
        {
 #ifndef WIN32
-               ACCEPT_TYPE_ARG3 size = sizeof(port->default_keepalives_idle);
+               socklen_t       size = sizeof(port->default_keepalives_idle);
 
                if (getsockopt(port->sock, IPPROTO_TCP, PG_TCP_KEEPALIVE_IDLE,
                                           (char *) &port->default_keepalives_idle,
@@ -1717,7 +1717,7 @@ pq_getkeepalivesinterval(Port *port)
        if (port->default_keepalives_interval == 0)
        {
 #ifndef WIN32
-               ACCEPT_TYPE_ARG3 size = sizeof(port->default_keepalives_interval);
+               socklen_t       size = sizeof(port->default_keepalives_interval);
 
                if (getsockopt(port->sock, IPPROTO_TCP, TCP_KEEPINTVL,
                                           (char *) &port->default_keepalives_interval,
@@ -1800,7 +1800,7 @@ pq_getkeepalivescount(Port *port)
 
        if (port->default_keepalives_count == 0)
        {
-               ACCEPT_TYPE_ARG3 size = sizeof(port->default_keepalives_count);
+               socklen_t       size = sizeof(port->default_keepalives_count);
 
                if (getsockopt(port->sock, IPPROTO_TCP, TCP_KEEPCNT,
                                           (char *) &port->default_keepalives_count,
@@ -1875,7 +1875,7 @@ pq_gettcpusertimeout(Port *port)
 
        if (port->default_tcp_user_timeout == 0)
        {
-               ACCEPT_TYPE_ARG3 size = sizeof(port->default_tcp_user_timeout);
+               socklen_t       size = sizeof(port->default_tcp_user_timeout);
 
                if (getsockopt(port->sock, IPPROTO_TCP, TCP_USER_TIMEOUT,
                                           (char *) &port->default_tcp_user_timeout,
index b7d0fbaefddb005cf756b9675dfbdf4bbe1c151a..8c166e5e161b8c5577de45894a18d91f36810ce2 100644 (file)
@@ -391,7 +391,7 @@ static void pgstat_recv_tempfile(PgStat_MsgTempFile *msg, int len);
 void
 pgstat_init(void)
 {
-       ACCEPT_TYPE_ARG3 alen;
+       socklen_t       alen;
        struct addrinfo *addrs = NULL,
                           *addr,
                                hints;
@@ -624,7 +624,7 @@ retry2:
        {
                int                     old_rcvbuf;
                int                     new_rcvbuf;
-               ACCEPT_TYPE_ARG3 rcvbufsize = sizeof(old_rcvbuf);
+               socklen_t       rcvbufsize = sizeof(old_rcvbuf);
 
                if (getsockopt(pgStatSock, SOL_SOCKET, SO_RCVBUF,
                                           (char *) &old_rcvbuf, &rcvbufsize) < 0)
index be9d9705744191d6ff0091adda34686fe4e2c5a6..1bcc189deeb23e882a9a6f359d7337b5738f6343 100644 (file)
@@ -62,7 +62,7 @@ struct sockaddr_storage
 typedef struct
 {
        struct sockaddr_storage addr;
-       ACCEPT_TYPE_ARG3 salen;
+       socklen_t       salen;
 } SockAddr;
 
 /* Configure the UNIX socket location for the well known port. */
index 15ffdd895aad9314077e937ce6437cac1cdcf698..ca3592465ede7b86ea416e74c262e5aeac6dd8fc 100644 (file)
@@ -1,17 +1,5 @@
 /* src/include/pg_config.h.in.  Generated from configure.ac by autoheader.  */
 
-/* Define to the type of arg 1 of 'accept' */
-#undef ACCEPT_TYPE_ARG1
-
-/* Define to the type of arg 2 of 'accept' */
-#undef ACCEPT_TYPE_ARG2
-
-/* Define to the type of arg 3 of 'accept' */
-#undef ACCEPT_TYPE_ARG3
-
-/* Define to the return type of 'accept' */
-#undef ACCEPT_TYPE_RETURN
-
 /* Define if building universal (internal helper macro) */
 #undef AC_APPLE_UNIVERSAL_BUILD
 
 /* Define to 1 if you have the `shm_open' function. */
 #undef HAVE_SHM_OPEN
 
+/* Define to 1 if the system has the type `socklen_t'. */
+#undef HAVE_SOCKLEN_T
+
 /* Define to 1 if you have spinlocks. */
 #undef HAVE_SPINLOCKS
 
index 2ff529fa59b687e1e6339c03517d96bdb6ee5e15..ae68d9c3c7938c9128fc0675d721c64f8a1f839f 100644 (file)
@@ -37,6 +37,10 @@ typedef SOCKET pgsocket;
 #define PGINVALID_SOCKET INVALID_SOCKET
 #endif
 
+#ifndef HAVE_SOCKLEN_T
+typedef int socklen_t;
+#endif
+
 /* non-blocking */
 extern bool pg_set_noblock(pgsocket sock);
 extern bool pg_set_block(pgsocket sock);
index f0fdd294a401da323fd5915bf5d62887e6ae91c6..9b6a6939f07a8154819eb082df24b0a315c1ff76 100644 (file)
@@ -2744,7 +2744,7 @@ keep_going:                                               /* We will come back to here until there is
 
                case CONNECTION_STARTED:
                        {
-                               ACCEPT_TYPE_ARG3 optlen = sizeof(optval);
+                               socklen_t       optlen = sizeof(optval);
 
                                /*
                                 * Write ready, since we've made it here, so the connection
index d6aa755d30a437fdbe74b28ecdc23d0a7fc04e90..4631869180714275415d9ffdf96cec9abee79afe 100644 (file)
@@ -37,7 +37,7 @@ getpeereid(int sock, uid_t *uid, gid_t *gid)
 #if defined(SO_PEERCRED)
        /* Linux: use getsockopt(SO_PEERCRED) */
        struct ucred peercred;
-       ACCEPT_TYPE_ARG3 so_len = sizeof(peercred);
+       socklen_t       so_len = sizeof(peercred);
 
        if (getsockopt(sock, SOL_SOCKET, SO_PEERCRED, &peercred, &so_len) != 0 ||
                so_len != sizeof(peercred))
@@ -48,7 +48,7 @@ getpeereid(int sock, uid_t *uid, gid_t *gid)
 #elif defined(LOCAL_PEERCRED)
        /* Debian with FreeBSD kernel: use getsockopt(LOCAL_PEERCRED) */
        struct xucred peercred;
-       ACCEPT_TYPE_ARG3 so_len = sizeof(peercred);
+       socklen_t       so_len = sizeof(peercred);
 
        if (getsockopt(sock, 0, LOCAL_PEERCRED, &peercred, &so_len) != 0 ||
                so_len != sizeof(peercred) ||
index 43fd1be08883277d9311c5a9bbab5b3b58992fc8..a013951e0d870795e7506574d226a7637fe7adf7 100644 (file)
@@ -205,10 +205,6 @@ sub GenerateFiles
        # Every symbol in pg_config.h.in must be accounted for here.  Set
        # to undef if the symbol should not be defined.
        my %define = (
-               ACCEPT_TYPE_ARG1           => 'unsigned int',
-               ACCEPT_TYPE_ARG2           => 'struct sockaddr *',
-               ACCEPT_TYPE_ARG3           => 'int',
-               ACCEPT_TYPE_RETURN         => 'unsigned int PASCAL',
                ALIGNOF_DOUBLE             => 8,
                ALIGNOF_INT                => 4,
                ALIGNOF_LONG               => 4,
@@ -365,6 +361,7 @@ sub GenerateFiles
                HAVE_SETPROCTITLE_FAST                   => undef,
                HAVE_SETSID                              => undef,
                HAVE_SHM_OPEN                            => undef,
+               HAVE_SOCKLEN_T                           => 1,
                HAVE_SPINLOCKS                           => 1,
                HAVE_SRANDOM                             => undef,
                HAVE_STDBOOL_H                           => 1,