(Writing): Add write, revise display.
[bpt/guile.git] / libguile / socket.c
index 2c5586b..6a0bc56 100644 (file)
@@ -1,50 +1,29 @@
-/* Copyright (C) 1996,1997,1998,2000,2001 Free Software Foundation, Inc.
+/* Copyright (C) 1996,1997,1998,2000,2001, 2002 Free Software Foundation, Inc.
  * 
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2, or (at your option)
- * any later version.
- * 
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- * 
- * You should have received a copy of the GNU General Public License
- * along with this software; see the file COPYING.  If not, write to
- * the Free Software Foundation, Inc., 59 Temple Place, Suite 330,
- * Boston, MA 02111-1307 USA
- *
- * As a special exception, the Free Software Foundation gives permission
- * for additional uses of the text contained in its release of GUILE.
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
  *
- * The exception is that, if you link the GUILE library with other files
- * to produce an executable, this does not by itself cause the
- * resulting executable to be covered by the GNU General Public License.
- * Your use of that executable is in no way restricted on account of
- * linking the GUILE library code into it.
- *
- * This exception does not however invalidate any other reasons why
- * the executable file might be covered by the GNU General Public License.
- *
- * This exception applies only to the code released by the
- * Free Software Foundation under the name GUILE.  If you copy
- * code from other Free Software Foundation releases into a copy of
- * GUILE, as the General Public License permits, the exception does
- * not apply to the code that you add in this way.  To avoid misleading
- * anyone as to the status of such modified files, you must delete
- * this exception notice from them.
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
  *
- * If you write modifications of your own for GUILE, it is your choice
- * whether to permit this exception to apply to your modifications.
- * If you do not wish that, delete this exception notice.  */
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
 
-/* Software engineering face-lift by Greg J. Badros, 11-Dec-1999,
-   gjb@cs.washington.edu, http://www.cs.washington.edu/homes/gjb */
 
 \f
 
+#if HAVE_CONFIG_H
+#  include <config.h>
+#endif
+
 #include <errno.h>
+#include <gmp.h>
 
 #include "libguile/_scm.h"
 #include "libguile/unif.h"
 #include "libguile/validate.h"
 #include "libguile/socket.h"
 
+#ifdef __MINGW32__
+#include "win32-socket.h"
+#endif
+
+#ifdef HAVE_STDINT_H
+#include <stdint.h>
+#endif
 #ifdef HAVE_STRING_H
 #include <string.h>
 #endif
@@ -63,6 +49,9 @@
 #include <unistd.h>
 #endif
 #include <sys/types.h>
+#ifdef HAVE_WINSOCK2_H
+#include <winsock2.h>
+#else
 #include <sys/socket.h>
 #ifdef HAVE_UNIX_DOMAIN_SOCKETS
 #include <sys/un.h>
@@ -70,6 +59,7 @@
 #include <netinet/in.h>
 #include <netdb.h>
 #include <arpa/inet.h>
+#endif
 
 #if defined (HAVE_UNIX_DOMAIN_SOCKETS) && !defined (SUN_LEN)
 #define SUN_LEN(ptr) ((size_t) (((struct sockaddr_un *) 0)->sun_path) \
 \f
 
 SCM_DEFINE (scm_htons, "htons", 1, 0, 0, 
-            (SCM in),
-           "Return a new integer from @var{value} by converting from host\n"
-           "to network order. @var{value} must be within the range of a C\n"
-           "unsigned short integer.")
+            (SCM value),
+           "Convert a 16 bit quantity from host to network byte ordering.\n"
+           "@var{value} is packed into 2 bytes, which are then converted\n"
+           "and returned as a new integer.")
 #define FUNC_NAME s_scm_htons
 {
   unsigned short c_in;
 
-  SCM_VALIDATE_INUM_COPY (1,in,c_in);
-  if (c_in != SCM_INUM (in))
-    SCM_OUT_OF_RANGE (1,in);
+  SCM_VALIDATE_INUM_COPY (1, value, c_in);
+  if (c_in != SCM_INUM (value))
+    SCM_OUT_OF_RANGE (1, value);
 
   return SCM_MAKINUM (htons (c_in));
 }
 #undef FUNC_NAME
 
 SCM_DEFINE (scm_ntohs, "ntohs", 1, 0, 0, 
-            (SCM in),
-           "Return a new integer from @var{value} by converting from\n"
-           "network to host order.  @var{value} must be within the range of\n"
-           "a C unsigned short integer.")
+            (SCM value),
+           "Convert a 16 bit quantity from network to host byte ordering.\n"
+           "@var{value} is packed into 2 bytes, which are then converted\n"
+           "and returned as a new integer.")
 #define FUNC_NAME s_scm_ntohs
 {
   unsigned short c_in;
 
-  SCM_VALIDATE_INUM_COPY (1,in,c_in);
-  if (c_in != SCM_INUM (in))
-    SCM_OUT_OF_RANGE (1,in);
+  SCM_VALIDATE_INUM_COPY (1, value, c_in);
+  if (c_in != SCM_INUM (value))
+    SCM_OUT_OF_RANGE (1, value);
 
   return SCM_MAKINUM (ntohs (c_in));
 }
 #undef FUNC_NAME
 
 SCM_DEFINE (scm_htonl, "htonl", 1, 0, 0, 
-            (SCM in),
-           "Return a new integer from @var{value} by converting from host\n"
-           "to network order. @var{value} must be within the range of a C\n"
-           "unsigned long integer.")
+            (SCM value),
+           "Convert a 32 bit quantity from host to network byte ordering.\n"
+           "@var{value} is packed into 4 bytes, which are then converted\n"
+           "and returned as a new integer.")
 #define FUNC_NAME s_scm_htonl
 {
-  unsigned long c_in = SCM_NUM2ULONG (1, in);
+  scm_t_uint32 c_in = SCM_NUM2ULONG (1, value);
+
   return scm_ulong2num (htonl (c_in));
 }
 #undef FUNC_NAME
 
 SCM_DEFINE (scm_ntohl, "ntohl", 1, 0, 0, 
-            (SCM in),
-           "Return a new integer from @var{value} by converting from\n"
-           "network to host order. @var{value} must be within the range of\n"
-           "a C unsigned long integer.")
+            (SCM value),
+           "Convert a 32 bit quantity from network to host byte ordering.\n"
+           "@var{value} is packed into 4 bytes, which are then converted\n"
+           "and returned as a new integer.")
 #define FUNC_NAME s_scm_ntohl
 {
-  unsigned long c_in = SCM_NUM2ULONG (1, in);
+  scm_t_uint32 c_in = SCM_NUM2ULONG (1, value);
+
   return scm_ulong2num (ntohl (c_in));
 }
 #undef FUNC_NAME
 
+#ifndef HAVE_INET_ATON
+/* for our definition in inet_aton.c, not usually needed.  */
+extern int inet_aton ();
+#endif
+
+SCM_DEFINE (scm_inet_aton, "inet-aton", 1, 0, 0, 
+            (SCM address),
+           "Convert an IPv4 Internet address from printable string\n"
+           "(dotted decimal notation) to an integer.  E.g.,\n\n"
+           "@lisp\n"
+           "(inet-aton \"127.0.0.1\") @result{} 2130706433\n"
+           "@end lisp")
+#define FUNC_NAME s_scm_inet_aton
+{
+  struct in_addr soka;
+
+  SCM_VALIDATE_STRING (1, address);
+  if (inet_aton (SCM_STRING_CHARS (address), &soka) == 0)
+    SCM_MISC_ERROR ("bad address", SCM_EOL);
+  return scm_ulong2num (ntohl (soka.s_addr));
+}
+#undef FUNC_NAME
+
+
+SCM_DEFINE (scm_inet_ntoa, "inet-ntoa", 1, 0, 0, 
+            (SCM inetid),
+           "Convert an IPv4 Internet address to a printable\n"
+           "(dotted decimal notation) string.  E.g.,\n\n"
+           "@lisp\n"
+           "(inet-ntoa 2130706433) @result{} \"127.0.0.1\"\n"
+           "@end lisp")
+#define FUNC_NAME s_scm_inet_ntoa
+{
+  struct in_addr addr;
+  char *s;
+  SCM answer;
+  addr.s_addr = htonl (SCM_NUM2ULONG (1, inetid));
+  s = inet_ntoa (addr);
+  answer = scm_mem2string (s, strlen (s));
+  return answer;
+}
+#undef FUNC_NAME
+
+#ifdef HAVE_INET_NETOF
+SCM_DEFINE (scm_inet_netof, "inet-netof", 1, 0, 0, 
+            (SCM address),
+           "Return the network number part of the given IPv4\n"
+           "Internet address.  E.g.,\n\n"
+           "@lisp\n"
+           "(inet-netof 2130706433) @result{} 127\n"
+           "@end lisp")
+#define FUNC_NAME s_scm_inet_netof
+{
+  struct in_addr addr;
+  addr.s_addr = htonl (SCM_NUM2ULONG (1, address));
+  return scm_ulong2num ((unsigned long) inet_netof (addr));
+}
+#undef FUNC_NAME
+#endif
+
+#ifdef HAVE_INET_LNAOF
+SCM_DEFINE (scm_lnaof, "inet-lnaof", 1, 0, 0, 
+            (SCM address),
+           "Return the local-address-with-network part of the given\n"
+           "IPv4 Internet address, using the obsolete class A/B/C system.\n"
+           "E.g.,\n\n"
+           "@lisp\n"
+           "(inet-lnaof 2130706433) @result{} 1\n"
+           "@end lisp")
+#define FUNC_NAME s_scm_lnaof
+{
+  struct in_addr addr;
+  addr.s_addr = htonl (SCM_NUM2ULONG (1, address));
+  return scm_ulong2num ((unsigned long) inet_lnaof (addr));
+}
+#undef FUNC_NAME
+#endif
+
+#ifdef HAVE_INET_MAKEADDR
+SCM_DEFINE (scm_inet_makeaddr, "inet-makeaddr", 2, 0, 0,
+            (SCM net, SCM lna),
+           "Make an IPv4 Internet address by combining the network number\n"
+           "@var{net} with the local-address-within-network number\n"
+           "@var{lna}.  E.g.,\n\n"
+           "@lisp\n"
+           "(inet-makeaddr 127 1) @result{} 2130706433\n"
+           "@end lisp")
+#define FUNC_NAME s_scm_inet_makeaddr
+{
+  struct in_addr addr;
+  unsigned long netnum;
+  unsigned long lnanum;
+
+  netnum = SCM_NUM2ULONG (1, net);
+  lnanum = SCM_NUM2ULONG (2, lna);
+  addr = inet_makeaddr (netnum, lnanum);
+  return scm_ulong2num (ntohl (addr.s_addr));
+}
+#undef FUNC_NAME
+#endif
+
+#ifdef HAVE_IPV6
+
+/* flip a 128 bit IPv6 address between host and network order.  */
+#ifdef WORDS_BIGENDIAN
+#define FLIP_NET_HOST_128(addr)
+#else
+#define FLIP_NET_HOST_128(addr)\
+{\
+  int i;\
+  \
+  for (i = 0; i < 8; i++)\
+    {\
+      scm_t_uint8 c = (addr)[i];\
+      \
+      (addr)[i] = (addr)[15 - i];\
+      (addr)[15 - i] = c;\
+    }\
+}
+#endif
+
+#ifdef WORDS_BIGENDIAN
+#define FLIPCPY_NET_HOST_128(dest, src) memcpy (dest, src, 16)
+#else
+#define FLIPCPY_NET_HOST_128(dest, src) \
+{ \
+  const scm_t_uint8 *tmp_srcp = (src) + 15; \
+  scm_t_uint8 *tmp_destp = (dest); \
+  \
+  do { \
+    *tmp_destp++ = *tmp_srcp--; \
+  } while (tmp_srcp != (src)); \
+}
+#endif
+
+
+#if (SIZEOF_SCM_T_BITS * SCM_CHAR_BIT) > 128
+#error "Assumption that scm_t_bits <= 128 bits has been violated."
+#endif
+
+#if (SIZEOF_UNSIGNED_LONG * SCM_CHAR_BIT) > 128
+#error "Assumption that unsigned long <= 128 bits has been violated."
+#endif
+
+#if (SIZEOF_UNSIGNED_LONG_LONG * SCM_CHAR_BIT) > 128
+#error "Assumption that unsigned long long <= 128 bits has been violated."
+#endif
+
+/* convert a 128 bit IPv6 address in network order to a host ordered
+   SCM integer.  */
+static SCM ipv6_net_to_num (const scm_t_uint8 *src)
+{
+  int i = 0;
+  const scm_t_uint8 *ptr = src;
+  int num_zero_bytes = 0;
+  scm_t_uint8 addr[16];
+
+  /* count leading zeros (since we know it's bigendian, they'll be first) */ 
+  while (i < 16)
+    {
+      if (*ptr) break;
+      num_zero_bytes++;
+      i++;
+    }
+
+  if (SCM_SIZEOF_UNSIGNED_LONG_LONG != 0) /* compiler should optimize this */
+    {
+      if ((16 - num_zero_bytes) <= sizeof (unsigned long long))
+        {
+          /* it fits */
+          unsigned long long x;
+          
+          FLIPCPY_NET_HOST_128(addr, src);
+#ifdef WORDS_BIGENDIAN
+          memcpy (&x, addr + (16 - sizeof (x)), sizeof (x));
+#else
+          memcpy (&x, addr, sizeof (x));
+#endif
+          return scm_ulong_long2num (x);
+        }
+    }
+  else
+    {
+      if ((16 - num_zero_bytes) <= sizeof (unsigned long))
+        {
+          /* this is just so that we use INUM where possible.  */
+          unsigned long x;
+
+          FLIPCPY_NET_HOST_128(addr, src);          
+#ifdef WORDS_BIGENDIAN
+          memcpy (&x, addr + (16 - sizeof (x)), sizeof (x));
+#else
+          memcpy (&x, addr, sizeof (x));
+#endif
+          return scm_ulong2num (x);
+        }
+    }
+  /* otherwise get the big hammer */
+  {
+    SCM result = scm_i_mkbig ();
+    
+    mpz_import (SCM_I_BIG_MPZ (result),
+                1, /* chunk */
+                1, /* big-endian chunk ordering */
+                16, /* chunks are 16 bytes long */
+                1, /* big-endian byte ordering */
+                0, /* "nails" -- leading unused bits per chunk */
+                src);
+    return scm_i_normbig (result);
+  }
+}  
+
+/* convert a host ordered SCM integer to a 128 bit IPv6 address in
+   network order.  */
+static void ipv6_num_to_net (SCM src, scm_t_uint8 *dst)
+{
+  /* This code presumes that src has already been checked for range. */
+  if (SCM_INUMP (src))
+    {
+      scm_t_signed_bits n = SCM_INUM (src);
+#ifdef WORDS_BIGENDIAN
+      memset (dst, 0, 16 - sizeof (scm_t_signed_bits));
+      memcpy (dst + (16 - sizeof (scm_t_signed_bits)),
+              &n,
+              sizeof (scm_t_signed_bits));
+#else
+      memset (dst + sizeof (scm_t_signed_bits),
+              0,
+              16 - sizeof (scm_t_signed_bits));
+      /* FIXME: this pair of ops is kinda wasteful -- should rewrite as
+         a single loop perhaps, similar to the handling of bignums. */
+      memcpy (dst, &n, sizeof (scm_t_signed_bits));
+      FLIP_NET_HOST_128 (dst);
+#endif
+    }
+  else
+    {
+      /* Presumes src has already been checked for fit -- see above. */
+      size_t count;
+      memset (dst, 0, 16);
+      mpz_export (dst,
+                  &count,
+                  1, /* big-endian chunk ordering */
+                  16, /* chunks are 16 bytes long */
+                  1, /* big-endian byte ordering */
+                  0, /* "nails" -- leading unused bits per chunk */
+                  SCM_I_BIG_MPZ (src));
+      scm_remember_upto_here_1 (src);
+    }
+}
+
+static int
+bignum_in_ipv6_range_p (SCM address)
+{
+  int result;
+  int sgn = mpz_sgn (SCM_I_BIG_MPZ (address));
+  
+  if (sgn < 0)
+    result = 0;
+  else
+    {
+      int size = mpz_sizeinbase (SCM_I_BIG_MPZ (address), 2);
+      if (size > 128) result = 0;
+      else result = 1;
+    }
+  scm_remember_upto_here_1 (address);
+  return result;
+}
+
+/* check that an SCM variable contains an IPv6 integer address.  */
+#define VALIDATE_INET6(which_arg, address)\
+   if (SCM_INUMP (address))\
+      SCM_ASSERT_RANGE (which_arg, address, SCM_INUM (address) >= 0);\
+   else\
+   {\
+      SCM_VALIDATE_BIGINT (which_arg, address);\
+      SCM_ASSERT_RANGE (which_arg, address, bignum_in_ipv6_range_p);\
+   }
+
+#ifdef HAVE_INET_PTON
+SCM_DEFINE (scm_inet_pton, "inet-pton", 2, 0, 0,
+            (SCM family, SCM address),
+           "Convert a string containing a printable network address to\n"
+           "an integer address.  Note that unlike the C version of this\n"
+           "function,\n"
+           "the result is an integer with normal host byte ordering.\n"
+           "@var{family} can be @code{AF_INET} or @code{AF_INET6}.  E.g.,\n\n"
+           "@lisp\n"
+           "(inet-pton AF_INET \"127.0.0.1\") @result{} 2130706433\n"
+           "(inet-pton AF_INET6 \"::1\") @result{} 1\n"
+           "@end lisp")
+#define FUNC_NAME s_scm_inet_pton
+{
+  int af;
+  char *src;
+  char dst[16];
+  int rv;
+
+  SCM_VALIDATE_INUM_COPY (1, family, af);
+  SCM_ASSERT_RANGE (1, family, af == AF_INET || af == AF_INET6);
+  SCM_VALIDATE_STRING_COPY (2, address, src);
+  rv = inet_pton (af, src, dst);
+  if (rv == -1)
+    SCM_SYSERROR;
+  else if (rv == 0)
+    SCM_MISC_ERROR ("Bad address", SCM_EOL);
+  if (af == AF_INET)
+    return scm_ulong2num (ntohl (*(scm_t_uint32 *) dst));
+  else
+    return ipv6_net_to_num ((char *) dst);
+}
+#undef FUNC_NAME
+#endif
+
+#ifdef HAVE_INET_NTOP
+SCM_DEFINE (scm_inet_ntop, "inet-ntop", 2, 0, 0,
+            (SCM family, SCM address),
+           "Convert a network address into a printable string.\n"
+           "Note that unlike the C version of this function,\n"
+           "the input is an integer with normal host byte ordering.\n"
+           "@var{family} can be @code{AF_INET} or @code{AF_INET6}.  E.g.,\n\n"
+           "@lisp\n"
+           "(inet-ntop AF_INET 2130706433) @result{} \"127.0.0.1\"\n"
+           "(inet-ntop AF_INET6 (- (expt 2 128) 1)) @result{}\n"
+           "ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff\n"
+           "@end lisp")
+#define FUNC_NAME s_scm_inet_ntop
+{
+  int af;
+#ifdef INET6_ADDRSTRLEN
+  char dst[INET6_ADDRSTRLEN];
+#else
+  char dst[46];
+#endif
+  char addr6[16];
+
+  SCM_VALIDATE_INUM_COPY (1, family, af);
+  SCM_ASSERT_RANGE (1, family, af == AF_INET || af == AF_INET6);
+  if (af == AF_INET)
+    *(scm_t_uint32 *) addr6 = htonl (SCM_NUM2ULONG (2, address));
+  else
+    {
+      VALIDATE_INET6 (2, address);
+      ipv6_num_to_net (address, addr6);
+    }
+  if (inet_ntop (af, &addr6, dst, sizeof dst) == NULL)
+    SCM_SYSERROR;
+  return scm_makfrom0str (dst);
+}
+#undef FUNC_NAME
+#endif
+
+#endif  /* HAVE_IPV6 */
+
 SCM_SYMBOL (sym_socket, "socket");
 
 #define SCM_SOCK_FD_TO_PORT(fd) scm_fdes_to_port (fd, "r+0", sym_socket)
@@ -147,16 +493,14 @@ SCM_SYMBOL (sym_socket, "socket");
 SCM_DEFINE (scm_socket, "socket", 3, 0, 0,
             (SCM family, SCM style, SCM proto),
            "Return a new socket port of the type specified by @var{family},\n"
-           "@var{style} and @var{protocol}.  All three parameters are\n"
+           "@var{style} and @var{proto}.  All three parameters are\n"
            "integers.  Supported values for @var{family} are\n"
            "@code{AF_UNIX}, @code{AF_INET} and @code{AF_INET6}.\n"
            "Typical values for @var{style} are @code{SOCK_STREAM},\n"
-           "@code{SOCK_DGRAM} and @code{SOCK_RAW}.\n"
-           "\n"
-           "@var{protocol} can be obtained from a protocol name using\n"
+           "@code{SOCK_DGRAM} and @code{SOCK_RAW}.\n\n"
+           "@var{proto} can be obtained from a protocol name using\n"
            "@code{getprotobyname}.  A value of zero specifies the default\n"
-           "protocol, which is usually right.\n"
-           "\n"
+           "protocol, which is usually right.\n\n"
            "A single socket port cannot by used for communication until it\n"
            "has been connected to another socket.")
 #define FUNC_NAME s_scm_socket
@@ -177,18 +521,18 @@ SCM_DEFINE (scm_socket, "socket", 3, 0, 0,
 SCM_DEFINE (scm_socketpair, "socketpair", 3, 0, 0,
             (SCM family, SCM style, SCM proto),
            "Return a pair of connected (but unnamed) socket ports of the\n"
-           "type specified by @var{family}, @var{style} and @var{protocol}.\n"
+           "type specified by @var{family}, @var{style} and @var{proto}.\n"
            "Many systems support only socket pairs of the @code{AF_UNIX}\n"
            "family.  Zero is likely to be the only meaningful value for\n"
-           "@var{protocol}.")
+           "@var{proto}.")
 #define FUNC_NAME s_scm_socketpair
 {
   int fam;
   int fd[2];
 
-  SCM_VALIDATE_INUM (1,family);
-  SCM_VALIDATE_INUM (2,style);
-  SCM_VALIDATE_INUM (3,proto);
+  SCM_VALIDATE_INUM (1, family);
+  SCM_VALIDATE_INUM (2, style);
+  SCM_VALIDATE_INUM (3, proto);
 
   fam = SCM_INUM (family);
 
@@ -203,12 +547,11 @@ SCM_DEFINE (scm_socketpair, "socketpair", 3, 0, 0,
 SCM_DEFINE (scm_getsockopt, "getsockopt", 3, 0, 0,
             (SCM sock, SCM level, SCM optname),
            "Return the value of a particular socket option for the socket\n"
-           "port @var{socket}.  @var{level} is an integer code for type of\n"
+           "port @var{sock}.  @var{level} is an integer code for type of\n"
            "option being requested, e.g., @code{SOL_SOCKET} for\n"
            "socket-level options.  @var{optname} is an integer code for the\n"
            "option required and should be specified using one of the\n"
-           "symbols @code{SO_DEBUG}, @code{SO_REUSEADDR} etc.\n"
-           "\n"
+           "symbols @code{SO_DEBUG}, @code{SO_REUSEADDR} etc.\n\n"
            "The returned value is typically an integer but @code{SO_LINGER}\n"
            "returns a pair of integers.")
 #define FUNC_NAME s_scm_getsockopt
@@ -219,8 +562,8 @@ SCM_DEFINE (scm_getsockopt, "getsockopt", 3, 0, 0,
   char optval[sizeof (struct linger)];
   int optlen = sizeof (struct linger);
 #else
-  char optval[sizeof (scm_sizet)];
-  int optlen = sizeof (scm_sizet);
+  char optval[sizeof (size_t)];
+  int optlen = sizeof (size_t);
 #endif
   int ilevel;
   int ioptname;
@@ -245,7 +588,7 @@ SCM_DEFINE (scm_getsockopt, "getsockopt", 3, 0, 0,
          return scm_cons (scm_long2num (ling->l_onoff),
                           scm_long2num (ling->l_linger));
 #else
-         return scm_cons (scm_long2num (*(int *) optval)
+         return scm_cons (scm_long2num (*(int *) optval),
                           SCM_MAKINUM (0));
 #endif
        }
@@ -260,7 +603,7 @@ SCM_DEFINE (scm_getsockopt, "getsockopt", 3, 0, 0,
 #endif
            )
          {
-           return scm_long2num (*(scm_sizet *) optval);
+           return scm_long2num (*(size_t *) optval);
          }
     }
   return scm_long2num (*(int *) optval);
@@ -269,8 +612,8 @@ SCM_DEFINE (scm_getsockopt, "getsockopt", 3, 0, 0,
 
 SCM_DEFINE (scm_setsockopt, "setsockopt", 4, 0, 0,
             (SCM sock, SCM level, SCM optname, SCM value),
-           "Sets the value of a particular socket option for the socket\n"
-           "port @var{socket}.  @var{level} is an integer code for type of option\n"
+           "Set the value of a particular socket option for the socket\n"
+           "port @var{sock}.  @var{level} is an integer code for type of option\n"
            "being set, e.g., @code{SOL_SOCKET} for socket-level options.\n"
            "@var{optname} is an\n"
            "integer code for the option to set and should be specified using one of\n"
@@ -287,7 +630,7 @@ SCM_DEFINE (scm_setsockopt, "setsockopt", 4, 0, 0,
 #ifdef HAVE_STRUCT_LINGER
   char optval[sizeof (struct linger)];
 #else
-  char optval[sizeof (scm_sizet)];
+  char optval[sizeof (size_t)];
 #endif
   int ilevel, ioptname;
 
@@ -346,8 +689,8 @@ SCM_DEFINE (scm_setsockopt, "setsockopt", 4, 0, 0,
          {
            long lv = SCM_NUM2LONG (4, value);
 
-           optlen = (int) sizeof (scm_sizet);
-           (*(scm_sizet *) optval) = (scm_sizet) lv;
+           optlen = (int) sizeof (size_t);
+           (*(size_t *) optval) = (size_t) lv;
          }
     }
   if (optlen == -1)
@@ -366,28 +709,10 @@ SCM_DEFINE (scm_setsockopt, "setsockopt", 4, 0, 0,
 }
 #undef FUNC_NAME
 
-/* flip a 128 bit IPv6 address between host and network order.  */
-#ifdef WORDS_BIGENDIAN
-#define FLIP_NET_HOST_128(addr)
-#else
-#define FLIP_NET_HOST_128(addr)\
-{\
-  int i;\
-  \
-  for (i = 0; i < 8; i++)\
-    {\
-      char c = (addr)[i];\
-      \
-      (addr)[i] = (addr)[15 - i];\
-      (addr)[15 - i] = c;\
-    }\
-}
-#endif
-
 SCM_DEFINE (scm_shutdown, "shutdown", 2, 0, 0,
           (SCM sock, SCM how),
            "Sockets can be closed simply by using @code{close-port}. The\n"
-           "@code{shutdown} procedure allows reception or tranmission on a\n"
+           "@code{shutdown} procedure allows reception or transmission on a\n"
            "connection to be shut down individually, according to the parameter\n"
            "@var{how}:\n\n"
            "@table @asis\n"
@@ -405,9 +730,9 @@ SCM_DEFINE (scm_shutdown, "shutdown", 2, 0, 0,
 {
   int fd;
   sock = SCM_COERCE_OUTPORT (sock);
-  SCM_VALIDATE_OPFPORT (1,sock);
-  SCM_VALIDATE_INUM (2,how);
-  SCM_ASSERT_RANGE(2,how,0 <= SCM_INUM (how) && 2 >= SCM_INUM (how));
+  SCM_VALIDATE_OPFPORT (1, sock);
+  SCM_VALIDATE_INUM (2, how);
+  SCM_ASSERT_RANGE(2, how,0 <= SCM_INUM (how) && 2 >= SCM_INUM (how));
   fd = SCM_FPORT_FDES (sock);
   if (shutdown (fd, SCM_INUM (how)) == -1)
     SCM_SYSERROR;
@@ -439,7 +764,7 @@ scm_fill_sockaddr (int fam, SCM address, SCM *args, int which_arg,
        SCM_VALIDATE_CONS (which_arg + 1, *args);
        SCM_VALIDATE_INUM_COPY (which_arg + 1, SCM_CAR (*args), port);
        *args = SCM_CDR (*args);
-       soka = (struct sockaddr_in *) malloc (sizeof (struct sockaddr_in));
+       soka = (struct sockaddr_in *) scm_malloc (sizeof (struct sockaddr_in));
        if (!soka)
          scm_memory_error (proc);
        /* 4.4BSD-style interface includes sin_len member and defines SIN_LEN,
@@ -453,7 +778,7 @@ scm_fill_sockaddr (int fam, SCM address, SCM *args, int which_arg,
        *size = sizeof (struct sockaddr_in);
        return (struct sockaddr *) soka;
       }
-#ifdef AF_INET6
+#ifdef HAVE_IPV6
     case AF_INET6:
       {
        /* see RFC2553.  */
@@ -462,16 +787,7 @@ scm_fill_sockaddr (int fam, SCM address, SCM *args, int which_arg,
        unsigned long flowinfo = 0;
        unsigned long scope_id = 0;
 
-       if (SCM_INUMP (address))
-         SCM_ASSERT_RANGE (which_arg, address, SCM_INUM (address) >= 0);
-       else
-         {
-           SCM_VALIDATE_BIGINT (which_arg, address);
-           SCM_ASSERT_RANGE (which_arg, address,
-                             !SCM_BIGSIGN (address)
-                             && (SCM_BITSPERDIG
-                                 * SCM_NUMDIGS (address) <= 128));
-         }
+       VALIDATE_INET6 (which_arg, address);
        SCM_VALIDATE_CONS (which_arg + 1, *args);
        SCM_VALIDATE_INUM_COPY (which_arg + 1, SCM_CAR (*args), port);
        *args = SCM_CDR (*args);
@@ -486,27 +802,14 @@ scm_fill_sockaddr (int fam, SCM address, SCM *args, int which_arg,
                *args = SCM_CDR (*args);
              }
          }
-       soka = (struct sockaddr_in6 *) malloc (sizeof (struct sockaddr_in6));
+       soka = (struct sockaddr_in6 *) scm_malloc (sizeof (struct sockaddr_in6));
        if (!soka)
          scm_memory_error (proc);
 #ifdef SIN_LEN6
        soka->sin6_len = sizeof (struct sockaddr_in6);
 #endif
        soka->sin6_family = AF_INET6;
-       if (SCM_INUMP (address))
-         {
-           uint32_t addr = htonl (SCM_INUM (address));
-
-           memset (soka->sin6_addr.s6_addr, 0, 12);
-           memcpy (soka->sin6_addr.s6_addr + 12, &addr, 4);
-         }
-       else
-         {
-           memset (soka->sin6_addr.s6_addr, 0, 16);
-           memcpy (soka->sin6_addr.s6_addr, SCM_BDIGITS (address), 
-                   SCM_NUMDIGS (address) * (SCM_BITSPERDIG / 8));
-           FLIP_NET_HOST_128 (soka->sin6_addr.s6_addr);
-         }
+       ipv6_num_to_net (address, soka->sin6_addr.s6_addr);
        soka->sin6_port = htons (port);
        soka->sin6_flowinfo = flowinfo;
 #ifdef HAVE_SIN6_SCOPE_ID
@@ -531,7 +834,7 @@ scm_fill_sockaddr (int fam, SCM address, SCM *args, int which_arg,
           member of the structure.  */
        addr_size = sizeof (struct sockaddr_un)
          + max (0, SCM_STRING_LENGTH (address) + 1 - (sizeof soka->sun_path));
-       soka = (struct sockaddr_un *) malloc (addr_size);
+       soka = (struct sockaddr_un *) scm_malloc (addr_size);
        if (!soka)
          scm_memory_error (proc);
        memset (soka, 0, addr_size);  /* for sun_len: see sin_len above. */
@@ -550,7 +853,7 @@ scm_fill_sockaddr (int fam, SCM address, SCM *args, int which_arg,
   
 SCM_DEFINE (scm_connect, "connect", 3, 0, 1,
             (SCM sock, SCM fam, SCM address, SCM args),
-           "Initiates a connection from a socket using a specified address\n"
+           "Initiate a connection from a socket using a specified address\n"
            "family to the address\n"
            "specified by @var{address} and possibly @var{args}.\n"
            "The format required for @var{address}\n"
@@ -574,8 +877,8 @@ SCM_DEFINE (scm_connect, "connect", 3, 0, 1,
   int size;
 
   sock = SCM_COERCE_OUTPORT (sock);
-  SCM_VALIDATE_OPFPORT (1,sock);
-  SCM_VALIDATE_INUM (2,fam);
+  SCM_VALIDATE_OPFPORT (1, sock);
+  SCM_VALIDATE_INUM (2, fam);
   fd = SCM_FPORT_FDES (sock);
   soka = scm_fill_sockaddr (SCM_INUM (fam), address, &args, 3, FUNC_NAME,
                            &size);
@@ -594,20 +897,21 @@ SCM_DEFINE (scm_connect, "connect", 3, 0, 1,
 
 SCM_DEFINE (scm_bind, "bind", 3, 0, 1,
             (SCM sock, SCM fam, SCM address, SCM args),
-           "Assigns an address to the socket port @var{socket}.\n"
+           "Assign an address to the socket port @var{sock}.\n"
            "Generally this only needs to be done for server sockets,\n"
            "so they know where to look for incoming connections.  A socket\n"
            "without an address will be assigned one automatically when it\n"
            "starts communicating.\n\n"
-           "The format of @var{address} and @var{ARG} @dots{} depends on the family\n"
-           "of the socket.\n\n"
+           "The format of @var{address} and @var{args} depends\n"
+           "on the family of the socket.\n\n"
            "For a socket of family @code{AF_UNIX}, only @var{address}\n"
-           "is specified and must \n"
-           "be a string with the filename where the socket is to be created.\n\n"
-           "For a socket of family @code{AF_INET}, @var{address} must be an integer\n"
-           "Internet host address and @var{arg} @dots{} must be a single integer\n"
-           "port number.\n\n"
-           "The values of the following variables can also be used for @var{address}:\n\n"
+           "is specified and must be a string with the filename where\n"
+           "the socket is to be created.\n\n"
+           "For a socket of family @code{AF_INET}, @var{address}\n"
+           "must be an integer IPv4 address and @var{args}\n"
+           "must be a single integer port number.\n\n"
+           "The values of the following variables can also be used for\n"
+           "@var{address}:\n\n"
            "@defvar INADDR_ANY\n"
            "Allow connections from any address.\n"
            "@end defvar\n\n"
@@ -620,6 +924,11 @@ SCM_DEFINE (scm_bind, "bind", 3, 0, 1,
            "@defvar INADDR_NONE\n"
            "No address.\n"
            "@end defvar\n\n"
+           "For a socket of family @code{AF_INET6}, @var{address}\n"
+           "must be an integer IPv6 address and @var{args}\n"
+           "may be up to three integers:\n"
+           "port [flowinfo] [scope_id],\n"
+           "where flowinfo and scope_id default to zero.\n\n"
            "The return value is unspecified.")
 #define FUNC_NAME s_scm_bind
 {
@@ -648,18 +957,19 @@ SCM_DEFINE (scm_bind, "bind", 3, 0, 1,
 
 SCM_DEFINE (scm_listen, "listen", 2, 0, 0,
             (SCM sock, SCM backlog),
-           "This procedure enables @var{socket} to accept connection\n"
+           "Enable @var{sock} to accept connection\n"
            "requests.  @var{backlog} is an integer specifying\n"
            "the maximum length of the queue for pending connections.\n"
-           "If the queue fills, new clients will fail to connect until the\n"
-           "server calls @code{accept} to accept a connection from the queue.\n\n"
+           "If the queue fills, new clients will fail to connect until\n"
+           "the server calls @code{accept} to accept a connection from\n"
+           "the queue.\n\n"
            "The return value is unspecified.")
 #define FUNC_NAME s_scm_listen
 {
   int fd;
   sock = SCM_COERCE_OUTPORT (sock);
-  SCM_VALIDATE_OPFPORT (1,sock);
-  SCM_VALIDATE_INUM (2,backlog);
+  SCM_VALIDATE_OPFPORT (1, sock);
+  SCM_VALIDATE_INUM (2, backlog);
   fd = SCM_FPORT_FDES (sock);
   if (listen (fd, SCM_INUM (backlog)) == -1)
     SCM_SYSERROR;
@@ -669,11 +979,12 @@ SCM_DEFINE (scm_listen, "listen", 2, 0, 0,
 
 /* Put the components of a sockaddr into a new SCM vector.  */
 static SCM
-scm_addr_vector (const struct sockaddr *address, const char *proc)
+scm_addr_vector (const struct sockaddr *address, int addr_size, 
+                const char *proc)
 {
   short int fam = address->sa_family;
-  SCM result;
-  SCM *ve;
+  SCM result =SCM_EOL;
+
 
   switch (fam)
     {
@@ -682,59 +993,26 @@ scm_addr_vector (const struct sockaddr *address, const char *proc)
        const struct sockaddr_in *nad = (struct sockaddr_in *) address;
 
        result = scm_c_make_vector (3, SCM_UNSPECIFIED);
-       ve = SCM_VELTS (result);
-       ve[0] = scm_ulong2num ((unsigned long) fam);
-       ve[1] = scm_ulong2num (ntohl (nad->sin_addr.s_addr));
-       ve[2] = scm_ulong2num ((unsigned long) ntohs (nad->sin_port));
+
+       SCM_VECTOR_SET(result, 0, scm_ulong2num ((unsigned long) fam));
+       SCM_VECTOR_SET(result, 1, scm_ulong2num (ntohl (nad->sin_addr.s_addr)));
+       SCM_VECTOR_SET(result, 2, scm_ulong2num ((unsigned long) ntohs (nad->sin_port)));
       }
       break;
-#ifdef AF_INET6
+#ifdef HAVE_IPV6
     case AF_INET6:
       {
        const struct sockaddr_in6 *nad = (struct sockaddr_in6 *) address;
-       int big_digits = 128 / SCM_BITSPERDIG;
-       int bytes_per_dig = SCM_BITSPERDIG / 8;
-       char addr[16];
-       char *ptr = addr;
-       SCM scm_addr;
-
-       memcpy (addr, nad->sin6_addr.s6_addr, 16);
-       /* get rid of leading zeros.  */ 
-       while (big_digits > 0)
-         {
-           long test = 0;
-
-           memcpy (&test, ptr, bytes_per_dig);
-           if (test != 0)
-             break;
-           ptr += bytes_per_dig;
-           big_digits--;
-         }
-       FLIP_NET_HOST_128 (addr);
-       if (big_digits * bytes_per_dig <= sizeof (unsigned long))
-         {
-           /* this is just so that we use INUM where possible.  */
-           unsigned long l_addr;
-
-           memcpy (&l_addr, addr, sizeof (unsigned long));
-           scm_addr = scm_ulong2num (l_addr);
-         }
-       else
-         {
-           scm_addr = scm_mkbig (big_digits, 0);
-           memcpy (SCM_BDIGITS (scm_addr), addr, big_digits * bytes_per_dig);
-         }
 
        result = scm_c_make_vector (5, SCM_UNSPECIFIED);
-       ve = SCM_VELTS (result);
-       ve[0] = scm_ulong2num ((unsigned long) fam);
-       ve[1] = scm_addr;
-       ve[2] = scm_ulong2num ((unsigned long) ntohs (nad->sin6_port));
-       ve[3] = scm_ulong2num ((unsigned long) nad->sin6_flowinfo);
+       SCM_VECTOR_SET(result, 0, scm_ulong2num ((unsigned long) fam));
+       SCM_VECTOR_SET(result, 1, ipv6_net_to_num (nad->sin6_addr.s6_addr));
+       SCM_VECTOR_SET(result, 2, scm_ulong2num ((unsigned long) ntohs (nad->sin6_port)));
+       SCM_VECTOR_SET(result, 3, scm_ulong2num ((unsigned long) nad->sin6_flowinfo));
 #ifdef HAVE_SIN6_SCOPE_ID
-       ve[4] = scm_ulong2num ((unsigned long) nad->sin6_scope_id);
+       SCM_VECTOR_SET(result, 4, scm_ulong2num ((unsigned long) nad->sin6_scope_id));
 #else
-       ve[4] = SCM_INUM0;
+       SCM_VECTOR_SET(result, 4, SCM_INUM0);
 #endif
       }
       break;
@@ -745,16 +1023,21 @@ scm_addr_vector (const struct sockaddr *address, const char *proc)
        const struct sockaddr_un *nad = (struct sockaddr_un *) address;
 
        result = scm_c_make_vector (2, SCM_UNSPECIFIED);
-       ve = SCM_VELTS (result);
-       ve[0] = scm_ulong2num ((unsigned long) fam);
-       ve[1] = scm_makfromstr (nad->sun_path,
-                               (scm_sizet) strlen (nad->sun_path), 0);
+
+       SCM_VECTOR_SET(result, 0, scm_ulong2num ((unsigned long) fam));
+       /* When addr_size is not enough to cover sun_path, do not try
+          to access it. */
+       if (addr_size <= offsetof (struct sockaddr_un, sun_path))
+         SCM_VECTOR_SET(result, 1, SCM_BOOL_F);
+       else
+         SCM_VECTOR_SET(result, 1, scm_mem2string (nad->sun_path,
+                                                   strlen (nad->sun_path)));
       }
       break;
 #endif
     default:
       scm_misc_error (proc, "Unrecognised address family: ~A",
-                     SCM_LIST1 (SCM_MAKINUM (fam)));
+                     scm_list_1 (SCM_MAKINUM (fam)));
     }
   return result;
 }
@@ -769,7 +1052,7 @@ scm_addr_vector (const struct sockaddr *address, const char *proc)
 #define MAX_SIZE_UN 0
 #endif
 
-#if defined (AF_INET6)
+#if defined (HAVE_IPV6)
 #define MAX_SIZE_IN6 sizeof (struct sockaddr_in6)
 #else
 #define MAX_SIZE_IN6 0
@@ -780,16 +1063,17 @@ scm_addr_vector (const struct sockaddr *address, const char *proc)
 
 SCM_DEFINE (scm_accept, "accept", 1, 0, 0, 
             (SCM sock),
-           "Accepts a connection on a bound, listening socket @var{socket}.  If there\n"
-           "are no pending connections in the queue, it waits until\n"
-           "one is available unless the non-blocking option has been set on the\n"
-           "socket.\n\n"
+           "Accept a connection on a bound, listening socket.\n"
+           "If there\n"
+           "are no pending connections in the queue, wait until\n"
+           "one is available unless the non-blocking option has been\n"
+           "set on the socket.\n\n"
            "The return value is a\n"
-           "pair in which the CAR is a new socket port for the connection and\n"
-           "the CDR is an object with address information about the client which\n"
-           "initiated the connection.\n\n"
-           "If the address is not available then the CDR will be an empty vector.\n\n"
-           "@var{socket} does not become part of the\n"
+           "pair in which the @emph{car} is a new socket port for the\n"
+           "connection and\n"
+           "the @emph{cdr} is an object with address information about the\n"
+           "client which initiated the connection.\n\n"
+           "@var{sock} does not become part of the\n"
            "connection and will continue to accept new requests.")
 #define FUNC_NAME s_scm_accept
 {
@@ -808,81 +1092,72 @@ SCM_DEFINE (scm_accept, "accept", 1, 0, 0,
   if (newfd == -1)
     SCM_SYSERROR;
   newsock = SCM_SOCK_FD_TO_PORT (newfd);
-  if (addr_size > 0)
-    address = scm_addr_vector (addr, FUNC_NAME);
-  else
-    address = SCM_BOOL_F;
-  
+  address = scm_addr_vector (addr, addr_size, FUNC_NAME);
   return scm_cons (newsock, address);
 }
 #undef FUNC_NAME
 
 SCM_DEFINE (scm_getsockname, "getsockname", 1, 0, 0, 
             (SCM sock),
-           "Return the address of @var{socket}, in the same form as the\n"
+           "Return the address of @var{sock}, in the same form as the\n"
            "object returned by @code{accept}.  On many systems the address\n"
            "of a socket in the @code{AF_FILE} namespace cannot be read.")
 #define FUNC_NAME s_scm_getsockname
 {
   int fd;
-  SCM result;
   int addr_size = MAX_ADDR_SIZE;
   char max_addr[MAX_ADDR_SIZE];
   struct sockaddr *addr = (struct sockaddr *) max_addr;
 
   sock = SCM_COERCE_OUTPORT (sock);
-  SCM_VALIDATE_OPFPORT (1,sock);
+  SCM_VALIDATE_OPFPORT (1, sock);
   fd = SCM_FPORT_FDES (sock);
   if (getsockname (fd, addr, &addr_size) == -1)
     SCM_SYSERROR;
-  if (addr_size > 0)
-    result = scm_addr_vector (addr, FUNC_NAME);
-  else
-    result = SCM_BOOL_F;
-  return result;
+  return scm_addr_vector (addr, addr_size, FUNC_NAME);
 }
 #undef FUNC_NAME
 
 SCM_DEFINE (scm_getpeername, "getpeername", 1, 0, 0, 
             (SCM sock),
-           "Return the address of the socket that the socket @var{socket}\n"
+           "Return the address that @var{sock}\n"
            "is connected to, in the same form as the object returned by\n"
            "@code{accept}.  On many systems the address of a socket in the\n"
            "@code{AF_FILE} namespace cannot be read.")
 #define FUNC_NAME s_scm_getpeername
 {
   int fd;
-  SCM result;
   int addr_size = MAX_ADDR_SIZE;
   char max_addr[MAX_ADDR_SIZE];
   struct sockaddr *addr = (struct sockaddr *) max_addr;
 
   sock = SCM_COERCE_OUTPORT (sock);
-  SCM_VALIDATE_OPFPORT (1,sock);
+  SCM_VALIDATE_OPFPORT (1, sock);
   fd = SCM_FPORT_FDES (sock);
   if (getpeername (fd, addr, &addr_size) == -1)
     SCM_SYSERROR;
-  if (addr_size > 0)
-    result = scm_addr_vector (addr, FUNC_NAME);
-  else
-    result = SCM_BOOL_F;
-  return result;
+  return scm_addr_vector (addr, addr_size, FUNC_NAME);
 }
 #undef FUNC_NAME
 
 SCM_DEFINE (scm_recv, "recv!", 2, 1, 0,
             (SCM sock, SCM buf, SCM flags),
-           "Receives data from the socket port @var{socket}.  @var{socket} must already\n"
+           "Receive data from a socket port.\n"
+           "@var{sock} must already\n"
            "be bound to the address from which data is to be received.\n"
            "@var{buf} is a string into which\n"
-           "the data will be written.  The size of @var{buf} limits the amount of\n"
+           "the data will be written.  The size of @var{buf} limits\n"
+           "the amount of\n"
            "data which can be received: in the case of packet\n"
-           "protocols, if a packet larger than this limit is encountered then some data\n"
+           "protocols, if a packet larger than this limit is encountered\n"
+           "then some data\n"
            "will be irrevocably lost.\n\n"
            "The optional @var{flags} argument is a value or\n"
            "bitwise OR of MSG_OOB, MSG_PEEK, MSG_DONTROUTE etc.\n\n"
-           "The value returned is the number of bytes read from the socket.\n\n"
-           "Note that the data is read directly from the socket file descriptor:\n"
+           "The value returned is the number of bytes read from the\n"
+           "socket.\n\n"
+           "Note that the data is read directly from the socket file\n"
+           "descriptor:\n"
            "any unread buffered port data is ignored.")
 #define FUNC_NAME s_scm_recv
 {
@@ -890,9 +1165,9 @@ SCM_DEFINE (scm_recv, "recv!", 2, 1, 0,
   int fd;
   int flg;
 
-  SCM_VALIDATE_OPFPORT (1,sock);
-  SCM_VALIDATE_STRING (2,buf);
-  SCM_VALIDATE_INUM_DEF_COPY (3,flags,0,flg);
+  SCM_VALIDATE_OPFPORT (1, sock);
+  SCM_VALIDATE_STRING (2, buf);
+  SCM_VALIDATE_INUM_DEF_COPY (3, flags,0, flg);
   fd = SCM_FPORT_FDES (sock);
 
   SCM_SYSCALL (rv = recv (fd, SCM_STRING_CHARS (buf), SCM_STRING_LENGTH (buf), flg));
@@ -905,13 +1180,17 @@ SCM_DEFINE (scm_recv, "recv!", 2, 1, 0,
 
 SCM_DEFINE (scm_send, "send", 2, 1, 0,
             (SCM sock, SCM message, SCM flags),
-           "Transmits the string @var{message} on the socket port @var{socket}. \n"
-           "@var{socket} must already be bound to a destination address.  The\n"
-           "value returned is the number of bytes transmitted -- it's possible for\n"
-           "this to be less than the length of @var{message} if the socket is\n"
-           "set to be non-blocking.  The optional @var{flags} argument is a value or\n"
+           "Transmit the string @var{message} on a socket port @var{sock}.\n"
+           "@var{sock} must already be bound to a destination address.  The\n"
+           "value returned is the number of bytes transmitted --\n"
+           "it's possible for\n"
+           "this to be less than the length of @var{message}\n"
+           "if the socket is\n"
+           "set to be non-blocking.  The optional @var{flags} argument\n"
+           "is a value or\n"
            "bitwise OR of MSG_OOB, MSG_PEEK, MSG_DONTROUTE etc.\n\n"
-           "Note that the data is written directly to the socket file descriptor:\n"
+           "Note that the data is written directly to the socket\n"
+           "file descriptor:\n"
            "any unflushed buffered port data is ignored.")
 #define FUNC_NAME s_scm_send
 {
@@ -920,9 +1199,9 @@ SCM_DEFINE (scm_send, "send", 2, 1, 0,
   int flg;
 
   sock = SCM_COERCE_OUTPORT (sock);
-  SCM_VALIDATE_OPFPORT (1,sock);
+  SCM_VALIDATE_OPFPORT (1, sock);
   SCM_VALIDATE_STRING (2, message);
-  SCM_VALIDATE_INUM_DEF_COPY (3,flags,0,flg);
+  SCM_VALIDATE_INUM_DEF_COPY (3, flags,0, flg);
   fd = SCM_FPORT_FDES (sock);
 
   SCM_SYSCALL (rv = send (fd, SCM_STRING_CHARS (message), SCM_STRING_LENGTH (message), flg));
@@ -934,25 +1213,23 @@ SCM_DEFINE (scm_send, "send", 2, 1, 0,
 
 SCM_DEFINE (scm_recvfrom, "recvfrom!", 2, 3, 0,
             (SCM sock, SCM str, SCM flags, SCM start, SCM end),
-           "Return data from the socket port @var{socket} and also\n"
+           "Return data from the socket port @var{sock} and also\n"
            "information about where the data was received from.\n"
-           "@var{socket} must already be bound to the address from which\n"
+           "@var{sock} must already be bound to the address from which\n"
            "data is to be received.  @code{str}, is a string into which the\n"
            "data will be written.  The size of @var{str} limits the amount\n"
            "of data which can be received: in the case of packet protocols,\n"
            "if a packet larger than this limit is encountered then some\n"
-           "data will be irrevocably lost.\n"
-           "\n"
+           "data will be irrevocably lost.\n\n"
            "The optional @var{flags} argument is a value or bitwise OR of\n"
-           "@code{MSG_OOB}, @code{MSG_PEEK}, @code{MSG_DONTROUTE} etc.\n"
-           "\n"
+           "@code{MSG_OOB}, @code{MSG_PEEK}, @code{MSG_DONTROUTE} etc.\n\n"
            "The value returned is a pair: the @emph{car} is the number of\n"
            "bytes read from the socket and the @emph{cdr} an address object\n"
-           "in the same form as returned by @code{accept}.\n"
-           "\n"
+           "in the same form as returned by @code{accept}.  The address\n"
+           "will given as @code{#f} if not available, as is usually the\n"
+           "case for stream sockets.\n\n"
            "The @var{start} and @var{end} arguments specify a substring of\n"
-           "@var{str} to which the data should be written.\n"
-           "\n"
+           "@var{str} to which the data should be written.\n\n"
            "Note that the data is read directly from the socket file\n"
            "descriptor: any unread buffered port data is ignored.")
 #define FUNC_NAME s_scm_recvfrom
@@ -968,7 +1245,7 @@ SCM_DEFINE (scm_recvfrom, "recvfrom!", 2, 3, 0,
   char max_addr[MAX_ADDR_SIZE];
   struct sockaddr *addr = (struct sockaddr *) max_addr;
 
-  SCM_VALIDATE_OPFPORT (1,sock);
+  SCM_VALIDATE_OPFPORT (1, sock);
   fd = SCM_FPORT_FDES (sock);
   SCM_VALIDATE_SUBSTRING_SPEC_COPY (2, str, buf, 4, start, offset,
                                    5, end, cend);
@@ -985,8 +1262,8 @@ SCM_DEFINE (scm_recvfrom, "recvfrom!", 2, 3, 0,
                              addr, &addr_size));
   if (rv == -1)
     SCM_SYSERROR;
-  if (addr_size > 0 && addr->sa_family != AF_UNSPEC)
-    address = scm_addr_vector (addr, FUNC_NAME);
+  if (addr->sa_family != AF_UNSPEC)
+    address = scm_addr_vector (addr, addr_size, FUNC_NAME);
   else
     address = SCM_BOOL_F;
 
@@ -996,15 +1273,22 @@ SCM_DEFINE (scm_recvfrom, "recvfrom!", 2, 3, 0,
 
 SCM_DEFINE (scm_sendto, "sendto", 4, 0, 1,
             (SCM sock, SCM message, SCM fam, SCM address, SCM args_and_flags),
-           "Transmits the string @var{message} on the socket port @var{socket}.  The\n"
-           "destination address is specified using the @var{family}, @var{address} and\n"
-           "@var{arg} arguments, in a similar way to the @code{connect}\n"
-           "procedure.  The\n"
-           "value returned is the number of bytes transmitted -- it's possible for\n"
-           "this to be less than the length of @var{message} if the socket is\n"
-           "set to be non-blocking.  The optional @var{flags} argument is a value or\n"
+           "Transmit the string @var{message} on the socket port\n"
+           "@var{sock}.  The\n"
+           "destination address is specified using the @var{fam},\n"
+           "@var{address} and\n"
+           "@var{args_and_flags} arguments, in a similar way to the\n"
+           "@code{connect} procedure.  @var{args_and_flags} contains\n"
+           "the usual connection arguments optionally followed by\n"
+           "a flags argument, which is a value or\n"
            "bitwise OR of MSG_OOB, MSG_PEEK, MSG_DONTROUTE etc.\n\n"
-           "Note that the data is written directly to the socket file descriptor:\n"
+           "The value returned is the number of bytes transmitted --\n"
+           "it's possible for\n"
+           "this to be less than the length of @var{message} if the\n"
+           "socket is\n"
+           "set to be non-blocking.\n"
+           "Note that the data is written directly to the socket\n"
+           "file descriptor:\n"
            "any unflushed buffered port data is ignored.")
 #define FUNC_NAME s_scm_sendto
 {
@@ -1015,9 +1299,9 @@ SCM_DEFINE (scm_sendto, "sendto", 4, 0, 1,
   int size;
 
   sock = SCM_COERCE_OUTPORT (sock);
-  SCM_VALIDATE_FPORT (1,sock);
+  SCM_VALIDATE_FPORT (1, sock);
   SCM_VALIDATE_STRING (2, message);
-  SCM_VALIDATE_INUM (3,fam);
+  SCM_VALIDATE_INUM (3, fam);
   fd = SCM_FPORT_FDES (sock);
   soka = scm_fill_sockaddr (SCM_INUM (fam), address, &args_and_flags, 4,
                            FUNC_NAME, &size);
@@ -1025,7 +1309,7 @@ SCM_DEFINE (scm_sendto, "sendto", 4, 0, 1,
     flg = 0;
   else
     {
-      SCM_VALIDATE_CONS (5,args_and_flags);
+      SCM_VALIDATE_CONS (5, args_and_flags);
       flg = SCM_NUM2ULONG (5, SCM_CAR (args_and_flags));
     }
   SCM_SYSCALL (rv = sendto (fd, SCM_STRING_CHARS (message),
@@ -1050,116 +1334,132 @@ scm_init_socket ()
 {
   /* protocol families.  */
 #ifdef AF_UNSPEC
-  scm_sysintern ("AF_UNSPEC", SCM_MAKINUM (AF_UNSPEC));
+  scm_c_define ("AF_UNSPEC", SCM_MAKINUM (AF_UNSPEC));
 #endif
 #ifdef AF_UNIX
-  scm_sysintern ("AF_UNIX", SCM_MAKINUM (AF_UNIX));
+  scm_c_define ("AF_UNIX", SCM_MAKINUM (AF_UNIX));
 #endif
 #ifdef AF_INET
-  scm_sysintern ("AF_INET", SCM_MAKINUM (AF_INET));
+  scm_c_define ("AF_INET", SCM_MAKINUM (AF_INET));
 #endif
 #ifdef AF_INET6
-  scm_sysintern ("AF_INET6", SCM_MAKINUM (AF_INET6));
+  scm_c_define ("AF_INET6", SCM_MAKINUM (AF_INET6));
 #endif
 
 #ifdef PF_UNSPEC
-  scm_sysintern ("PF_UNSPEC", SCM_MAKINUM (PF_UNSPEC));
+  scm_c_define ("PF_UNSPEC", SCM_MAKINUM (PF_UNSPEC));
 #endif
 #ifdef PF_UNIX
-  scm_sysintern ("PF_UNIX", SCM_MAKINUM (PF_UNIX));
+  scm_c_define ("PF_UNIX", SCM_MAKINUM (PF_UNIX));
 #endif
 #ifdef PF_INET
-  scm_sysintern ("PF_INET", SCM_MAKINUM (PF_INET));
+  scm_c_define ("PF_INET", SCM_MAKINUM (PF_INET));
 #endif
 #ifdef PF_INET6
-  scm_sysintern ("PF_INET6", SCM_MAKINUM (PF_INET6));
+  scm_c_define ("PF_INET6", SCM_MAKINUM (PF_INET6));
+#endif
+
+  /* standard addresses.  */
+#ifdef INADDR_ANY
+  scm_c_define ("INADDR_ANY", scm_ulong2num (INADDR_ANY));
+#endif
+#ifdef INADDR_BROADCAST
+  scm_c_define ("INADDR_BROADCAST", scm_ulong2num (INADDR_BROADCAST));
+#endif
+#ifdef INADDR_NONE
+  scm_c_define ("INADDR_NONE", scm_ulong2num (INADDR_NONE));
+#endif
+#ifdef INADDR_LOOPBACK
+  scm_c_define ("INADDR_LOOPBACK", scm_ulong2num (INADDR_LOOPBACK));
 #endif
 
   /* socket types.  */
 #ifdef SOCK_STREAM
-  scm_sysintern ("SOCK_STREAM", SCM_MAKINUM (SOCK_STREAM));
+  scm_c_define ("SOCK_STREAM", SCM_MAKINUM (SOCK_STREAM));
 #endif
 #ifdef SOCK_DGRAM
-  scm_sysintern ("SOCK_DGRAM", SCM_MAKINUM (SOCK_DGRAM));
+  scm_c_define ("SOCK_DGRAM", SCM_MAKINUM (SOCK_DGRAM));
 #endif
 #ifdef SOCK_RAW
-  scm_sysintern ("SOCK_RAW", SCM_MAKINUM (SOCK_RAW));
+  scm_c_define ("SOCK_RAW", SCM_MAKINUM (SOCK_RAW));
 #endif
 
   /* setsockopt level.  */
 #ifdef SOL_SOCKET
-  scm_sysintern ("SOL_SOCKET", SCM_MAKINUM (SOL_SOCKET));
+  scm_c_define ("SOL_SOCKET", SCM_MAKINUM (SOL_SOCKET));
 #endif
 #ifdef SOL_IP
-  scm_sysintern ("SOL_IP", SCM_MAKINUM (SOL_IP));
+  scm_c_define ("SOL_IP", SCM_MAKINUM (SOL_IP));
 #endif
 #ifdef SOL_TCP
-  scm_sysintern ("SOL_TCP", SCM_MAKINUM (SOL_TCP));
+  scm_c_define ("SOL_TCP", SCM_MAKINUM (SOL_TCP));
 #endif
 #ifdef SOL_UDP
-  scm_sysintern ("SOL_UDP", SCM_MAKINUM (SOL_UDP));
+  scm_c_define ("SOL_UDP", SCM_MAKINUM (SOL_UDP));
 #endif
 
   /* setsockopt names.  */
 #ifdef SO_DEBUG
-  scm_sysintern ("SO_DEBUG", SCM_MAKINUM (SO_DEBUG));
+  scm_c_define ("SO_DEBUG", SCM_MAKINUM (SO_DEBUG));
 #endif
 #ifdef SO_REUSEADDR
-  scm_sysintern ("SO_REUSEADDR", SCM_MAKINUM (SO_REUSEADDR));
+  scm_c_define ("SO_REUSEADDR", SCM_MAKINUM (SO_REUSEADDR));
 #endif
 #ifdef SO_STYLE
-  scm_sysintern ("SO_STYLE", SCM_MAKINUM (SO_STYLE));
+  scm_c_define ("SO_STYLE", SCM_MAKINUM (SO_STYLE));
 #endif
 #ifdef SO_TYPE
-  scm_sysintern ("SO_TYPE", SCM_MAKINUM (SO_TYPE));
+  scm_c_define ("SO_TYPE", SCM_MAKINUM (SO_TYPE));
 #endif
 #ifdef SO_ERROR
-  scm_sysintern ("SO_ERROR", SCM_MAKINUM (SO_ERROR));
+  scm_c_define ("SO_ERROR", SCM_MAKINUM (SO_ERROR));
 #endif
 #ifdef SO_DONTROUTE
-  scm_sysintern ("SO_DONTROUTE", SCM_MAKINUM (SO_DONTROUTE));
+  scm_c_define ("SO_DONTROUTE", SCM_MAKINUM (SO_DONTROUTE));
 #endif
 #ifdef SO_BROADCAST
-  scm_sysintern ("SO_BROADCAST", SCM_MAKINUM (SO_BROADCAST));
+  scm_c_define ("SO_BROADCAST", SCM_MAKINUM (SO_BROADCAST));
 #endif
 #ifdef SO_SNDBUF
-  scm_sysintern ("SO_SNDBUF", SCM_MAKINUM (SO_SNDBUF));
+  scm_c_define ("SO_SNDBUF", SCM_MAKINUM (SO_SNDBUF));
 #endif
 #ifdef SO_RCVBUF
-  scm_sysintern ("SO_RCVBUF", SCM_MAKINUM (SO_RCVBUF));
+  scm_c_define ("SO_RCVBUF", SCM_MAKINUM (SO_RCVBUF));
 #endif
 #ifdef SO_KEEPALIVE
-  scm_sysintern ("SO_KEEPALIVE", SCM_MAKINUM (SO_KEEPALIVE));
+  scm_c_define ("SO_KEEPALIVE", SCM_MAKINUM (SO_KEEPALIVE));
 #endif
 #ifdef SO_OOBINLINE
-  scm_sysintern ("SO_OOBINLINE", SCM_MAKINUM (SO_OOBINLINE));
+  scm_c_define ("SO_OOBINLINE", SCM_MAKINUM (SO_OOBINLINE));
 #endif
 #ifdef SO_NO_CHECK
-  scm_sysintern ("SO_NO_CHECK", SCM_MAKINUM (SO_NO_CHECK));
+  scm_c_define ("SO_NO_CHECK", SCM_MAKINUM (SO_NO_CHECK));
 #endif
 #ifdef SO_PRIORITY
-  scm_sysintern ("SO_PRIORITY", SCM_MAKINUM (SO_PRIORITY));
+  scm_c_define ("SO_PRIORITY", SCM_MAKINUM (SO_PRIORITY));
 #endif
 #ifdef SO_LINGER
-  scm_sysintern ("SO_LINGER", SCM_MAKINUM (SO_LINGER));
+  scm_c_define ("SO_LINGER", SCM_MAKINUM (SO_LINGER));
 #endif
 
   /* recv/send options.  */
 #ifdef MSG_OOB
-  scm_sysintern ("MSG_OOB", SCM_MAKINUM (MSG_OOB));
+  scm_c_define ("MSG_OOB", SCM_MAKINUM (MSG_OOB));
 #endif
 #ifdef MSG_PEEK
-  scm_sysintern ("MSG_PEEK", SCM_MAKINUM (MSG_PEEK));
+  scm_c_define ("MSG_PEEK", SCM_MAKINUM (MSG_PEEK));
 #endif
 #ifdef MSG_DONTROUTE
-  scm_sysintern ("MSG_DONTROUTE", SCM_MAKINUM (MSG_DONTROUTE));
+  scm_c_define ("MSG_DONTROUTE", SCM_MAKINUM (MSG_DONTROUTE));
+#endif
+
+#ifdef __MINGW32__
+  scm_i_init_socket_Win32 ();
 #endif
 
   scm_add_feature ("socket");
 
-#ifndef SCM_MAGIC_SNARFER
 #include "libguile/socket.x"
-#endif
 }