merge from 1.8 branch
[bpt/guile.git] / libguile / posix.c
index 1745903..8a83a1e 100644 (file)
@@ -1,4 +1,4 @@
-/* Copyright (C) 1995,1996,1997,1998,1999,2000,2001, 2002, 2003, 2004 Free Software Foundation, Inc.
+/* Copyright (C) 1995,1996,1997,1998,1999,2000,2001, 2002, 2003, 2004, 2005, 2006 Free Software Foundation, Inc.
  * 
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -12,7 +12,7 @@
  *
  * 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
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
 
 #include <errno.h>
 
 #include "libguile/_scm.h"
+#include "libguile/dynwind.h"
 #include "libguile/fports.h"
 #include "libguile/scmsigs.h"
 #include "libguile/feature.h"
 #include "libguile/strings.h"
+#include "libguile/srfi-13.h"
+#include "libguile/srfi-14.h"
 #include "libguile/vectors.h"
 #include "libguile/lang.h"
 
 #include "libguile/validate.h"
 #include "libguile/posix.h"
+#include "libguile/i18n.h"
+#include "libguile/threads.h"
 \f
 
 #ifdef HAVE_STRING_H
@@ -114,6 +119,14 @@ extern char ** environ;
 #  include <crypt.h>
 #endif
 
+#ifdef HAVE_NETDB_H
+#include <netdb.h>      /* for MAXHOSTNAMELEN on Solaris */
+#endif
+
+#ifdef HAVE_SYS_PARAM_H
+#include <sys/param.h>  /* for MAXHOSTNAMELEN */
+#endif
+
 #if HAVE_SYS_RESOURCE_H
 #  include <sys/resource.h>
 #endif
@@ -144,6 +157,12 @@ extern char ** environ;
 #define F_OK 0
 #endif
 
+/* No prototype for this on Solaris 10.  The man page says it's in
+   <unistd.h> ... but it lies. */
+#if ! HAVE_DECL_SETHOSTNAME
+int sethostname (char *name, size_t namelen);
+#endif
+
 /* On NextStep, <utime.h> doesn't define struct utime, unless we
    #define _POSIX_SOURCE before #including it.  I think this is less
    of a kludge than defining struct utimbuf ourselves.  */
@@ -180,6 +199,27 @@ extern char ** environ;
 #endif
 
 \f
+
+/* Two often used patterns
+ */
+
+#define WITH_STRING(str,cstr,code)             \
+  do {                                         \
+    char *cstr = scm_to_locale_string (str);   \
+    code;                                      \
+    free (cstr);                               \
+  } while (0)
+
+#define STRING_SYSCALL(str,cstr,code)        \
+  do {                                       \
+    int eno;                                 \
+    char *cstr = scm_to_locale_string (str); \
+    SCM_SYSCALL (code);                      \
+    eno = errno; free (cstr); errno = eno;   \
+  } while (0)
+
+
+\f
 SCM_SYMBOL (sym_read_pipe, "read pipe");
 SCM_SYMBOL (sym_write_pipe, "write pipe");
 
@@ -236,7 +276,7 @@ SCM_DEFINE (scm_getgroups, "getgroups", 0, 0, 0,
 
   result = scm_c_make_vector (ngroups, SCM_BOOL_F);
   while (--ngroups >= 0) 
-    SCM_VECTOR_SET (result, ngroups, scm_ulong2num (groups[ngroups]));
+    SCM_SIMPLE_VECTOR_SET (result, ngroups, scm_from_ulong (groups[ngroups]));
 
   free (groups);
   return result;
@@ -247,7 +287,11 @@ SCM_DEFINE (scm_getgroups, "getgroups", 0, 0, 0,
 #ifdef HAVE_SETGROUPS
 SCM_DEFINE (scm_setgroups, "setgroups", 1, 0, 0,
             (SCM group_vec),
-           "Set the supplementary group IDs to those found in the vector argument.")
+           "Set the current set of supplementary group IDs to the integers\n"
+           "in the given vector @var{vec}.  The return value is\n"
+           "unspecified.\n"
+           "\n"
+           "Generally only the superuser can set the process group IDs.")
 #define FUNC_NAME s_scm_setgroups
 {
   size_t ngroups;
@@ -259,25 +303,26 @@ SCM_DEFINE (scm_setgroups, "setgroups", 1, 0, 0,
 
   SCM_VALIDATE_VECTOR (SCM_ARG1, group_vec);
 
-  ngroups = SCM_VECTOR_LENGTH (group_vec);
+  ngroups = SCM_SIMPLE_VECTOR_LENGTH (group_vec);
 
   /* validate before allocating, so we don't have to worry about leaks */
   for (i = 0; i < ngroups; i++)
     {
       unsigned long ulong_gid;
       GETGROUPS_T gid;
-      SCM_VALIDATE_ULONG_COPY (1, SCM_VECTOR_REF (group_vec, i), ulong_gid);
+      SCM_VALIDATE_ULONG_COPY (1, SCM_SIMPLE_VECTOR_REF (group_vec, i),
+                              ulong_gid);
       gid = ulong_gid;
       if (gid != ulong_gid)
-       SCM_OUT_OF_RANGE (1, SCM_VECTOR_REF (group_vec, i));
+       SCM_OUT_OF_RANGE (1, SCM_SIMPLE_VECTOR_REF (group_vec, i));
     }
 
   size = ngroups * sizeof (GETGROUPS_T);
   if (size / sizeof (GETGROUPS_T) != ngroups)
-    SCM_OUT_OF_RANGE (SCM_ARG1, SCM_MAKINUM (ngroups));
+    SCM_OUT_OF_RANGE (SCM_ARG1, scm_from_int (ngroups));
   groups = scm_malloc (size);
   for(i = 0; i < ngroups; i++)
-    groups [i] = SCM_NUM2ULONG (1, SCM_VECTOR_REF (group_vec, i));
+    groups [i] = SCM_NUM2ULONG (1, SCM_SIMPLE_VECTOR_REF (group_vec, i));
 
   result = setgroups (ngroups, groups);
   save_errno = errno; /* don't let free() touch errno */
@@ -301,7 +346,7 @@ SCM_DEFINE (scm_getpwuid, "getpw", 0, 1, 0,
   struct passwd *entry;
 
   SCM result = scm_c_make_vector (7, SCM_UNSPECIFIED);
-  if (SCM_UNBNDP (user) || SCM_FALSEP (user))
+  if (SCM_UNBNDP (user) || scm_is_false (user))
     {
       SCM_SYSCALL (entry = getpwent ());
       if (! entry)
@@ -309,31 +354,31 @@ SCM_DEFINE (scm_getpwuid, "getpw", 0, 1, 0,
          return SCM_BOOL_F;
        }
     }
-  else if (SCM_INUMP (user))
+  else if (scm_is_integer (user))
     {
-      entry = getpwuid (SCM_INUM (user));
+      entry = getpwuid (scm_to_int (user));
     }
   else
     {
-      SCM_VALIDATE_STRING (1, user);
-      entry = getpwnam (SCM_STRING_CHARS (user));
+      WITH_STRING (user, c_user,
+                  entry = getpwnam (c_user));
     }
   if (!entry)
     SCM_MISC_ERROR ("entry not found", SCM_EOL);
 
-  SCM_VECTOR_SET(result, 0, scm_makfrom0str (entry->pw_name));
-  SCM_VECTOR_SET(result, 1, scm_makfrom0str (entry->pw_passwd));
-  SCM_VECTOR_SET(result, 2, scm_ulong2num ((unsigned long) entry->pw_uid));
-  SCM_VECTOR_SET(result, 3, scm_ulong2num ((unsigned long) entry->pw_gid));
-  SCM_VECTOR_SET(result, 4, scm_makfrom0str (entry->pw_gecos));
+  SCM_SIMPLE_VECTOR_SET(result, 0, scm_from_locale_string (entry->pw_name));
+  SCM_SIMPLE_VECTOR_SET(result, 1, scm_from_locale_string (entry->pw_passwd));
+  SCM_SIMPLE_VECTOR_SET(result, 2, scm_from_ulong (entry->pw_uid));
+  SCM_SIMPLE_VECTOR_SET(result, 3, scm_from_ulong (entry->pw_gid));
+  SCM_SIMPLE_VECTOR_SET(result, 4, scm_from_locale_string (entry->pw_gecos));
   if (!entry->pw_dir)
-    SCM_VECTOR_SET(result, 5, scm_makfrom0str (""));
+    SCM_SIMPLE_VECTOR_SET(result, 5, scm_from_locale_string (""));
   else
-    SCM_VECTOR_SET(result, 5, scm_makfrom0str (entry->pw_dir));
+    SCM_SIMPLE_VECTOR_SET(result, 5, scm_from_locale_string (entry->pw_dir));
   if (!entry->pw_shell)
-    SCM_VECTOR_SET(result, 6, scm_makfrom0str (""));
+    SCM_SIMPLE_VECTOR_SET(result, 6, scm_from_locale_string (""));
   else
-    SCM_VECTOR_SET(result, 6, scm_makfrom0str (entry->pw_shell));
+    SCM_SIMPLE_VECTOR_SET(result, 6, scm_from_locale_string (entry->pw_shell));
   return result;
 }
 #undef FUNC_NAME
@@ -348,7 +393,7 @@ SCM_DEFINE (scm_setpwent, "setpw", 0, 1, 0,
            "@code{endpwent} procedures are implemented on top of this.")
 #define FUNC_NAME s_scm_setpwent
 {
-  if (SCM_UNBNDP (arg) || SCM_FALSEP (arg))
+  if (SCM_UNBNDP (arg) || scm_is_false (arg))
     endpwent ();
   else
     setpwent ();
@@ -370,7 +415,7 @@ SCM_DEFINE (scm_getgrgid, "getgr", 0, 1, 0,
   struct group *entry;
   SCM result = scm_c_make_vector (4, SCM_UNSPECIFIED);
 
-  if (SCM_UNBNDP (name) || SCM_FALSEP (name))
+  if (SCM_UNBNDP (name) || scm_is_false (name))
     {
       SCM_SYSCALL (entry = getgrent ());
       if (! entry)
@@ -378,20 +423,18 @@ SCM_DEFINE (scm_getgrgid, "getgr", 0, 1, 0,
          return SCM_BOOL_F;
        }
     }
-  else if (SCM_INUMP (name))
-    SCM_SYSCALL (entry = getgrgid (SCM_INUM (name)));
+  else if (scm_is_integer (name))
+    SCM_SYSCALL (entry = getgrgid (scm_to_int (name)));
   else
-    {
-      SCM_VALIDATE_STRING (1, name);
-      SCM_SYSCALL (entry = getgrnam (SCM_STRING_CHARS (name)));
-    }
+    STRING_SYSCALL (name, c_name,
+                   entry = getgrnam (c_name));
   if (!entry)
     SCM_SYSERROR;
 
-  SCM_VECTOR_SET(result, 0, scm_makfrom0str (entry->gr_name));
-  SCM_VECTOR_SET(result, 1, scm_makfrom0str (entry->gr_passwd));
-  SCM_VECTOR_SET(result, 2, scm_ulong2num ((unsigned long) entry->gr_gid));
-  SCM_VECTOR_SET(result, 3, scm_makfromstrs (-1, entry->gr_mem));
+  SCM_SIMPLE_VECTOR_SET(result, 0, scm_from_locale_string (entry->gr_name));
+  SCM_SIMPLE_VECTOR_SET(result, 1, scm_from_locale_string (entry->gr_passwd));
+  SCM_SIMPLE_VECTOR_SET(result, 2, scm_from_ulong  (entry->gr_gid));
+  SCM_SIMPLE_VECTOR_SET(result, 3, scm_makfromstrs (-1, entry->gr_mem));
   return result;
 }
 #undef FUNC_NAME
@@ -405,7 +448,7 @@ SCM_DEFINE (scm_setgrent, "setgr", 0, 1, 0,
            "@code{endgrent} procedures are implemented on top of this.")
 #define FUNC_NAME s_scm_setgrent
 {
-  if (SCM_UNBNDP (arg) || SCM_FALSEP (arg))
+  if (SCM_UNBNDP (arg) || scm_is_false (arg))
     endgrent ();
   else
     setgrent ();
@@ -441,14 +484,12 @@ SCM_DEFINE (scm_kill, "kill", 2, 0, 0,
            "@end defvar")
 #define FUNC_NAME s_scm_kill
 {
-  SCM_VALIDATE_INUM (1, pid);
-  SCM_VALIDATE_INUM (2, sig);
   /* Signal values are interned in scm_init_posix().  */
 #ifdef HAVE_KILL
-  if (kill ((int) SCM_INUM (pid), (int) SCM_INUM (sig)) != 0)
+  if (kill (scm_to_int (pid), scm_to_int  (sig)) != 0)
 #else
-  if ((int) SCM_INUM (pid) == getpid ())
-    if (raise ((int) SCM_INUM (sig)) != 0)
+  if (scm_to_int (pid) == getpid ())
+    if (raise (scm_to_int (sig)) != 0)
 #endif
       SCM_SYSERROR;
   return SCM_UNSPECIFIED;
@@ -497,19 +538,17 @@ SCM_DEFINE (scm_waitpid, "waitpid", 1, 1, 0,
   int i;
   int status;
   int ioptions;
-  SCM_VALIDATE_INUM (1, pid);
   if (SCM_UNBNDP (options))
     ioptions = 0;
   else
     {
-      SCM_VALIDATE_INUM (2, options);
       /* Flags are interned in scm_init_posix.  */
-      ioptions = SCM_INUM (options);
+      ioptions = scm_to_int (options);
     }
-  SCM_SYSCALL (i = waitpid (SCM_INUM (pid), &status, ioptions));
+  SCM_SYSCALL (i = waitpid (scm_to_int (pid), &status, ioptions));
   if (i == -1)
     SCM_SYSERROR;
-  return scm_cons (SCM_MAKINUM (0L + i), SCM_MAKINUM (0L + status));
+  return scm_cons (scm_from_int (i), scm_from_int (status));
 }
 #undef FUNC_NAME
 #endif /* HAVE_WAITPID */
@@ -524,13 +563,11 @@ SCM_DEFINE (scm_status_exit_val, "status:exit-val", 1, 0, 0,
 {
   int lstatus;
 
-  SCM_VALIDATE_INUM (1, status);
-
   /* On Ultrix, the WIF... macros assume their argument is an lvalue;
-     go figure.  SCM_INUM does not yield an lvalue.  */
-  lstatus = SCM_INUM (status);
+     go figure.  */
+  lstatus = scm_to_int (status);
   if (WIFEXITED (lstatus))
-    return (SCM_MAKINUM (WEXITSTATUS (lstatus)));
+    return (scm_from_int (WEXITSTATUS (lstatus)));
   else
     return SCM_BOOL_F;
 }
@@ -544,11 +581,9 @@ SCM_DEFINE (scm_status_term_sig, "status:term-sig", 1, 0, 0,
 {
   int lstatus;
 
-  SCM_VALIDATE_INUM (1, status);
-
-  lstatus = SCM_INUM (status);
+  lstatus = scm_to_int (status);
   if (WIFSIGNALED (lstatus))
-    return SCM_MAKINUM (WTERMSIG (lstatus));
+    return scm_from_int (WTERMSIG (lstatus));
   else
     return SCM_BOOL_F;
 }
@@ -562,11 +597,9 @@ SCM_DEFINE (scm_status_stop_sig, "status:stop-sig", 1, 0, 0,
 {
   int lstatus;
 
-  SCM_VALIDATE_INUM (1, status);
-
-  lstatus = SCM_INUM (status);
+  lstatus = scm_to_int (status);
   if (WIFSTOPPED (lstatus))
-    return SCM_MAKINUM (WSTOPSIG (lstatus));
+    return scm_from_int (WSTOPSIG (lstatus));
   else
     return SCM_BOOL_F;
 }
@@ -580,7 +613,7 @@ SCM_DEFINE (scm_getppid, "getppid", 0, 0, 0,
            "process.")
 #define FUNC_NAME s_scm_getppid
 {
-  return SCM_MAKINUM (0L + getppid ());
+  return scm_from_int (getppid ());
 }
 #undef FUNC_NAME
 #endif /* HAVE_GETPPID */
@@ -592,7 +625,7 @@ SCM_DEFINE (scm_getuid, "getuid", 0, 0, 0,
            "Return an integer representing the current real user ID.")
 #define FUNC_NAME s_scm_getuid
 {
-  return SCM_MAKINUM (0L + getuid ());
+  return scm_from_int (getuid ());
 }
 #undef FUNC_NAME
 
@@ -603,7 +636,7 @@ SCM_DEFINE (scm_getgid, "getgid", 0, 0, 0,
            "Return an integer representing the current real group ID.")
 #define FUNC_NAME s_scm_getgid
 {
-  return SCM_MAKINUM (0L + getgid ());
+  return scm_from_int (getgid ());
 }
 #undef FUNC_NAME
 
@@ -618,9 +651,9 @@ SCM_DEFINE (scm_geteuid, "geteuid", 0, 0, 0,
 #define FUNC_NAME s_scm_geteuid
 {
 #ifdef HAVE_GETEUID
-  return SCM_MAKINUM (0L + geteuid ());
+  return scm_from_int (geteuid ());
 #else
-  return SCM_MAKINUM (0L + getuid ());
+  return scm_from_int (getuid ());
 #endif
 }
 #undef FUNC_NAME
@@ -635,9 +668,9 @@ SCM_DEFINE (scm_getegid, "getegid", 0, 0, 0,
 #define FUNC_NAME s_scm_getegid
 {
 #ifdef HAVE_GETEUID
-  return SCM_MAKINUM (0L + getegid ());
+  return scm_from_int (getegid ());
 #else
-  return SCM_MAKINUM (0L + getgid ());
+  return scm_from_int (getgid ());
 #endif
 }
 #undef FUNC_NAME
@@ -650,8 +683,7 @@ SCM_DEFINE (scm_setuid, "setuid", 1, 0, 0,
            "The return value is unspecified.")
 #define FUNC_NAME s_scm_setuid
 {
-  SCM_VALIDATE_INUM (1, id);
-  if (setuid (SCM_INUM (id)) != 0)
+  if (setuid (scm_to_int (id)) != 0)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
 }
@@ -664,8 +696,7 @@ SCM_DEFINE (scm_setgid, "setgid", 1, 0, 0,
            "The return value is unspecified.")
 #define FUNC_NAME s_scm_setgid
 {
-  SCM_VALIDATE_INUM (1, id);
-  if (setgid (SCM_INUM (id)) != 0)
+  if (setgid (scm_to_int (id)) != 0)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
 }
@@ -682,11 +713,10 @@ SCM_DEFINE (scm_seteuid, "seteuid", 1, 0, 0,
 {
   int rv;
 
-  SCM_VALIDATE_INUM (1, id);
 #ifdef HAVE_SETEUID
-  rv = seteuid (SCM_INUM (id));
+  rv = seteuid (scm_to_int (id));
 #else
-  rv = setuid (SCM_INUM (id));
+  rv = setuid (scm_to_int (id));
 #endif
   if (rv != 0)
     SCM_SYSERROR;
@@ -708,11 +738,10 @@ SCM_DEFINE (scm_setegid, "setegid", 1, 0, 0,
 {
   int rv;
 
-  SCM_VALIDATE_INUM (1, id);
 #ifdef HAVE_SETEUID
-  rv = setegid (SCM_INUM (id));
+  rv = setegid (scm_to_int (id));
 #else
-  rv = setgid (SCM_INUM (id));
+  rv = setgid (scm_to_int (id));
 #endif
   if (rv != 0)
     SCM_SYSERROR;
@@ -732,7 +761,7 @@ SCM_DEFINE (scm_getpgrp, "getpgrp", 0, 0, 0,
 {
   int (*fn)();
   fn = (int (*) ()) getpgrp;
-  return SCM_MAKINUM (fn (0));
+  return scm_from_int (fn (0));
 }
 #undef FUNC_NAME
 #endif /* HAVE_GETPGRP */
@@ -748,10 +777,8 @@ SCM_DEFINE (scm_setpgid, "setpgid", 2, 0, 0,
            "The return value is unspecified.")
 #define FUNC_NAME s_scm_setpgid
 {
-  SCM_VALIDATE_INUM (1, pid);
-  SCM_VALIDATE_INUM (2, pgid);
   /* FIXME(?): may be known as setpgrp.  */
-  if (setpgid (SCM_INUM (pid), SCM_INUM (pgid)) != 0)
+  if (setpgid (scm_to_int (pid), scm_to_int (pgid)) != 0)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
 }
@@ -775,6 +802,15 @@ SCM_DEFINE (scm_setsid, "setsid", 0, 0, 0,
 #undef FUNC_NAME
 #endif /* HAVE_SETSID */
 
+
+/* ttyname returns its result in a single static buffer, hence
+   scm_i_misc_mutex for thread safety.  In glibc 2.3.2 two threads
+   continuously calling ttyname will otherwise get an overwrite quite
+   easily.
+
+   ttyname_r (when available) could be used instead of scm_i_misc_mutex, but
+   there's probably little to be gained in either speed or parallelism.  */
+
 #ifdef HAVE_TTYNAME
 SCM_DEFINE (scm_ttyname, "ttyname", 1, 0, 0, 
             (SCM port),
@@ -783,22 +819,37 @@ SCM_DEFINE (scm_ttyname, "ttyname", 1, 0, 0,
 #define FUNC_NAME s_scm_ttyname
 {
   char *result;
-  int fd;
+  int fd, err;
+  SCM ret;
 
   port = SCM_COERCE_OUTPORT (port);
   SCM_VALIDATE_OPPORT (1, port);
   if (!SCM_FPORTP (port))
     return SCM_BOOL_F;
   fd = SCM_FPORT_FDES (port);
+
+  scm_i_scm_pthread_mutex_lock (&scm_i_misc_mutex);
   SCM_SYSCALL (result = ttyname (fd));
+  err = errno;
+  ret = scm_from_locale_string (result);
+  scm_i_pthread_mutex_unlock (&scm_i_misc_mutex);
+
   if (!result)
-    SCM_SYSERROR;
-  /* result could be overwritten by another call to ttyname */
-  return (scm_makfrom0str (result));
+    {
+      errno = err;
+      SCM_SYSERROR;
+    }
+  return ret;
 }
 #undef FUNC_NAME
 #endif /* HAVE_TTYNAME */
 
+
+/* For thread safety "buf" is used instead of NULL for the ctermid static
+   buffer.  Actually it's unlikely the controlling terminal will change
+   during program execution, and indeed on glibc (2.3.2) it's always just
+   "/dev/tty", but L_ctermid on the stack is easy and fast and guarantees
+   safety everywhere.  */
 #ifdef HAVE_CTERMID
 SCM_DEFINE (scm_ctermid, "ctermid", 0, 0, 0,
             (),
@@ -806,10 +857,11 @@ SCM_DEFINE (scm_ctermid, "ctermid", 0, 0, 0,
            "terminal for the current process.")
 #define FUNC_NAME s_scm_ctermid
 {
-  char *result = ctermid (NULL);
+  char buf[L_ctermid];
+  char *result = ctermid (buf);
   if (*result == '\0')
     SCM_SYSERROR;
-  return scm_makfrom0str (result);
+  return scm_from_locale_string (result);
 }
 #undef FUNC_NAME
 #endif /* HAVE_CTERMID */
@@ -838,7 +890,7 @@ SCM_DEFINE (scm_tcgetpgrp, "tcgetpgrp", 1, 0, 0,
   fd = SCM_FPORT_FDES (port);
   if ((pgid = tcgetpgrp (fd)) == -1)
     SCM_SYSERROR;
-  return SCM_MAKINUM (pgid);
+  return scm_from_int (pgid);
 }
 #undef FUNC_NAME    
 #endif /* HAVE_TCGETPGRP */
@@ -858,41 +910,18 @@ SCM_DEFINE (scm_tcsetpgrp, "tcsetpgrp", 2, 0, 0,
   port = SCM_COERCE_OUTPORT (port);
 
   SCM_VALIDATE_OPFPORT (1, port);
-  SCM_VALIDATE_INUM (2, pgid);
   fd = SCM_FPORT_FDES (port);
-  if (tcsetpgrp (fd, SCM_INUM (pgid)) == -1)
+  if (tcsetpgrp (fd, scm_to_int (pgid)) == -1)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
 }
 #undef FUNC_NAME
 #endif /* HAVE_TCSETPGRP */
 
-/* return a newly allocated array of char pointers to each of the strings
-   in args, with a terminating NULL pointer.  */
-/* Note: a similar function is defined in dynl.c, but we don't necessarily
-   want to export it.  */
-static char **allocate_string_pointers (SCM args)
+static void
+free_string_pointers (void *data)
 {
-  char **result;
-  int n_args = scm_ilength (args);
-  int i;
-
-  SCM_ASSERT (n_args >= 0, args, SCM_ARGn, "allocate_string_pointers");
-  result = (char **) scm_malloc ((n_args + 1) * sizeof (char *));
-  result[n_args] = NULL;
-  for (i = 0; i < n_args; i++)
-    {
-      SCM car = SCM_CAR (args);
-
-      if (!SCM_STRINGP (car))
-       {
-         free (result);
-         scm_wrong_type_arg ("allocate_string_pointers", SCM_ARGn, car);
-       }
-      result[i] = SCM_STRING_CHARS (SCM_CAR (args));
-      args = SCM_CDR (args);
-    }
-  return result;
+  scm_i_free_string_pointers ((char **)data);
 }
 
 SCM_DEFINE (scm_execl, "execl", 1, 0, 1, 
@@ -908,16 +937,28 @@ SCM_DEFINE (scm_execl, "execl", 1, 0, 1,
            "call, but we call it @code{execl} because of its Scheme calling interface.")
 #define FUNC_NAME s_scm_execl
 {
-  char **execargv;
-  int save_errno;
-  SCM_VALIDATE_STRING (1, filename);
-  execargv = allocate_string_pointers (args);
-  execv (SCM_STRING_CHARS (filename), execargv);
-  save_errno = errno;
-  free (execargv);
-  errno = save_errno;
+  char *exec_file;
+  char **exec_argv;
+
+  scm_dynwind_begin (0);
+
+  exec_file = scm_to_locale_string (filename);
+  scm_dynwind_free (exec_file);
+
+  exec_argv = scm_i_allocate_string_pointers (args);
+  scm_dynwind_unwind_handler (free_string_pointers, exec_argv, 
+                           SCM_F_WIND_EXPLICITLY);
+
+  execv (exec_file,
+#ifdef __MINGW32__
+         /* extra "const" in mingw formals, provokes warning from gcc */
+         (const char * const *)
+#endif
+         exec_argv);
   SCM_SYSERROR;
+
   /* not reached.  */
+  scm_dynwind_end ();
   return SCM_BOOL_F;
 }
 #undef FUNC_NAME
@@ -932,50 +973,32 @@ SCM_DEFINE (scm_execlp, "execlp", 1, 0, 1,
            "call, but we call it @code{execlp} because of its Scheme calling interface.")
 #define FUNC_NAME s_scm_execlp
 {
-  char **execargv;
-  int save_errno;
-  SCM_VALIDATE_STRING (1, filename);
-  execargv = allocate_string_pointers (args);
-  execvp (SCM_STRING_CHARS (filename), execargv);
-  save_errno = errno;
-  free (execargv);
-  errno = save_errno;
+  char *exec_file;
+  char **exec_argv;
+
+  scm_dynwind_begin (0);
+
+  exec_file = scm_to_locale_string (filename);
+  scm_dynwind_free (exec_file);
+
+  exec_argv = scm_i_allocate_string_pointers (args);
+  scm_dynwind_unwind_handler (free_string_pointers, exec_argv, 
+                           SCM_F_WIND_EXPLICITLY);
+
+  execvp (exec_file,
+#ifdef __MINGW32__
+          /* extra "const" in mingw formals, provokes warning from gcc */
+          (const char * const *)
+#endif
+          exec_argv);
   SCM_SYSERROR;
+
   /* not reached.  */
+  scm_dynwind_end ();
   return SCM_BOOL_F;
 }
 #undef FUNC_NAME
 
-static char **
-environ_list_to_c (SCM envlist, int arg, const char *proc)
-{
-  int num_strings;
-  char **result;
-  int i;
-
-  num_strings = scm_ilength (envlist);
-  SCM_ASSERT (num_strings >= 0, envlist, arg, proc);
-  result = (char **) scm_malloc ((num_strings + 1) * sizeof (char *));
-  if (result == NULL)
-    scm_memory_error (proc);
-  for (i = 0; !SCM_NULL_OR_NIL_P (envlist); ++i, envlist = SCM_CDR (envlist))
-    {
-      SCM str = SCM_CAR (envlist);
-      int len;
-      char *src;
-
-      SCM_ASSERT (SCM_STRINGP (str), envlist, arg, proc);
-      len = SCM_STRING_LENGTH (str);
-      src = SCM_STRING_CHARS (str);
-      result[i] = scm_malloc (len + 1);
-      if (result[i] == NULL)
-       scm_memory_error (proc);
-      memcpy (result[i], src, len);
-      result[i][len] = 0;
-    }
-  result[i] = 0;
-  return result;
-}
 
 /* OPTIMIZE-ME: scm_execle doesn't need malloced copies of the environment
    list strings the way environ_list_to_c gives.  */
@@ -989,23 +1012,38 @@ SCM_DEFINE (scm_execle, "execle", 2, 0, 1,
            "call, but we call it @code{execle} because of its Scheme calling interface.")
 #define FUNC_NAME s_scm_execle
 {
-  char **execargv;
+  char **exec_argv;
   char **exec_env;
-  int save_errno, i;
+  char *exec_file;
 
-  SCM_VALIDATE_STRING (1, filename);
-  
-  execargv = allocate_string_pointers (args);
-  exec_env = environ_list_to_c (env, SCM_ARG2, FUNC_NAME);
-  execve (SCM_STRING_CHARS (filename), execargv, exec_env);
-  save_errno = errno;
-  free (execargv);
-  for (i = 0; exec_env[i] != NULL; i++)
-    free (exec_env[i]);
-  free (exec_env);
-  errno = save_errno;
+  scm_dynwind_begin (0);
+
+  exec_file = scm_to_locale_string (filename);
+  scm_dynwind_free (exec_file);
+
+  exec_argv = scm_i_allocate_string_pointers (args);
+  scm_dynwind_unwind_handler (free_string_pointers, exec_argv,
+                           SCM_F_WIND_EXPLICITLY);
+
+  exec_env = scm_i_allocate_string_pointers (env);
+  scm_dynwind_unwind_handler (free_string_pointers, exec_env,
+                           SCM_F_WIND_EXPLICITLY);
+
+  execve (exec_file,
+#ifdef __MINGW32__
+          /* extra "const" in mingw formals, provokes warning from gcc */
+          (const char * const *)
+#endif
+          exec_argv,
+#ifdef __MINGW32__
+          /* extra "const" in mingw formals, provokes warning from gcc */
+          (const char * const *)
+#endif
+          exec_env);
   SCM_SYSERROR;
+
   /* not reached.  */
+  scm_dynwind_end ();
   return SCM_BOOL_F;
 }
 #undef FUNC_NAME
@@ -1024,7 +1062,7 @@ SCM_DEFINE (scm_fork, "primitive-fork", 0, 0, 0,
   pid = fork ();
   if (pid == -1)
     SCM_SYSERROR;
-  return SCM_MAKINUM (0L+pid);
+  return scm_from_int (pid);
 }
 #undef FUNC_NAME
 #endif /* HAVE_FORK */
@@ -1044,14 +1082,14 @@ SCM_DEFINE (scm_uname, "uname", 0, 0, 0,
   SCM result = scm_c_make_vector (5, SCM_UNSPECIFIED);
   if (uname (&buf) < 0)
     SCM_SYSERROR;
-  SCM_VECTOR_SET(result, 0, scm_makfrom0str (buf.sysname));
-  SCM_VECTOR_SET(result, 1, scm_makfrom0str (buf.nodename));
-  SCM_VECTOR_SET(result, 2, scm_makfrom0str (buf.release));
-  SCM_VECTOR_SET(result, 3, scm_makfrom0str (buf.version));
-  SCM_VECTOR_SET(result, 4, scm_makfrom0str (buf.machine));
+  SCM_SIMPLE_VECTOR_SET(result, 0, scm_from_locale_string (buf.sysname));
+  SCM_SIMPLE_VECTOR_SET(result, 1, scm_from_locale_string (buf.nodename));
+  SCM_SIMPLE_VECTOR_SET(result, 2, scm_from_locale_string (buf.release));
+  SCM_SIMPLE_VECTOR_SET(result, 3, scm_from_locale_string (buf.version));
+  SCM_SIMPLE_VECTOR_SET(result, 4, scm_from_locale_string (buf.machine));
 /* 
    a linux special?
-  SCM_VECTOR_SET(result, 5, scm_makfrom0str (buf.domainname));
+  SCM_SIMPLE_VECTOR_SET(result, 5, scm_from_locale_string (buf.domainname));
 */
   return result;
 }
@@ -1075,19 +1113,14 @@ SCM_DEFINE (scm_environ, "environ", 0, 1, 0,
     {
       char **new_environ;
 
-      new_environ = environ_list_to_c (env, SCM_ARG1, FUNC_NAME);
+      new_environ = scm_i_allocate_string_pointers (env);
       /* Free the old environment, except when called for the first
        * time.
        */
       {
-       char **ep;
        static int first = 1;
        if (!first)
-         {
-           for (ep = environ; *ep != NULL; ep++)
-             free (*ep);
-           free ((char *) environ);
-         }
+         scm_i_free_string_pointers (environ);
        first = 0;
       }
       environ = new_environ;
@@ -1114,7 +1147,7 @@ SCM_DEFINE (scm_tmpnam, "tmpnam", 0, 0, 0,
   if (rv == NULL)
     /* not SCM_SYSERROR since errno probably not set.  */
     SCM_MISC_ERROR ("tmpnam failed", SCM_EOL);
-  return scm_makfrom0str (name);
+  return scm_from_locale_string (name);
 }
 #undef FUNC_NAME
 
@@ -1126,20 +1159,44 @@ extern int mkstemp (char *);
 
 SCM_DEFINE (scm_mkstemp, "mkstemp!", 1, 0, 0,
            (SCM tmpl),
-           "Create a new unique file in the file system and returns a new\n"
+           "Create a new unique file in the file system and return a new\n"
            "buffered port open for reading and writing to the file.\n"
+           "\n"
            "@var{tmpl} is a string specifying where the file should be\n"
-           "created: it must end with @code{XXXXXX} and will be changed in\n"
-           "place to return the name of the temporary file.")
+           "created: it must end with @samp{XXXXXX} and those @samp{X}s\n"
+           "will be changed in the string to return the name of the file.\n"
+           "(@code{port-filename} on the port also gives the name.)\n"
+           "\n"
+           "POSIX doesn't specify the permissions mode of the file, on GNU\n"
+           "and most systems it's @code{#o600}.  An application can use\n"
+           "@code{chmod} to relax that if desired.  For example\n"
+           "@code{#o666} less @code{umask}, which is usual for ordinary\n"
+           "file creation,\n"
+           "\n"
+           "@example\n"
+           "(let ((port (mkstemp! (string-copy \"/tmp/myfile-XXXXXX\"))))\n"
+           "  (chmod port (logand #o666 (lognot (umask))))\n"
+           "  ...)\n"
+           "@end example")
 #define FUNC_NAME s_scm_mkstemp
 {
   char *c_tmpl;
   int rv;
   
-  SCM_VALIDATE_STRING_COPY (1, tmpl, c_tmpl);
+  scm_dynwind_begin (0);
+
+  c_tmpl = scm_to_locale_string (tmpl);
+  scm_dynwind_free (c_tmpl);
+
   SCM_SYSCALL (rv = mkstemp (c_tmpl));
   if (rv == -1)
     SCM_SYSERROR;
+
+  scm_substring_move_x (scm_from_locale_string (c_tmpl),
+                       SCM_INUM0, scm_string_length (tmpl),
+                       tmpl, SCM_INUM0);
+
+  scm_dynwind_end ();
   return scm_fdes_to_port (rv, "w+", tmpl);
 }
 #undef FUNC_NAME
@@ -1161,7 +1218,6 @@ SCM_DEFINE (scm_utime, "utime", 1, 2, 0,
   int rv;
   struct utimbuf utm_tmp;
 
-  SCM_VALIDATE_STRING (1, pathname);
   if (SCM_UNBNDP (actime))
     SCM_SYSCALL (time (&utm_tmp.actime));
   else
@@ -1172,7 +1228,8 @@ SCM_DEFINE (scm_utime, "utime", 1, 2, 0,
   else
     utm_tmp.modtime = SCM_NUM2ULONG (3, modtime);
 
-  SCM_SYSCALL (rv = utime (SCM_STRING_CHARS (pathname), &utm_tmp));
+  STRING_SYSCALL (pathname, c_pathname,
+                 rv = utime (c_pathname, &utm_tmp));
   if (rv != 0)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
@@ -1181,37 +1238,53 @@ SCM_DEFINE (scm_utime, "utime", 1, 2, 0,
 
 SCM_DEFINE (scm_access, "access?", 2, 0, 0,
             (SCM path, SCM how),
-           "Return @code{#t} if @var{path} corresponds to an existing file\n"
-           "and the current process has the type of access specified by\n"
-           "@var{how}, otherwise @code{#f}.  @var{how} should be specified\n"
-           "using the values of the variables listed below.  Multiple\n"
-           "values can be combined using a bitwise or, in which case\n"
-           "@code{#t} will only be returned if all accesses are granted.\n"
+           "Test accessibility of a file under the real UID and GID of the\n"
+           "calling process.  The return is @code{#t} if @var{path} exists\n"
+           "and the permissions requested by @var{how} are all allowed, or\n"
+           "@code{#f} if not.\n"
            "\n"
-           "Permissions are checked using the real id of the current\n"
-           "process, not the effective id, although it's the effective id\n"
-           "which determines whether the access would actually be granted.\n"
+           "@var{how} is an integer which is one of the following values,\n"
+           "or a bitwise-OR (@code{logior}) of multiple values.\n"
            "\n"
            "@defvar R_OK\n"
-           "test for read permission.\n"
+           "Test for read permission.\n"
            "@end defvar\n"
            "@defvar W_OK\n"
-           "test for write permission.\n"
+           "Test for write permission.\n"
            "@end defvar\n"
            "@defvar X_OK\n"
-           "test for execute permission.\n"
+           "Test for execute permission.\n"
            "@end defvar\n"
            "@defvar F_OK\n"
-           "test for existence of the file.\n"
-           "@end defvar")
+           "Test for existence of the file.  This is implied by each of the\n"
+           "other tests, so there's no need to combine it with them.\n"
+           "@end defvar\n"
+           "\n"
+           "It's important to note that @code{access?} does not simply\n"
+           "indicate what will happen on attempting to read or write a\n"
+           "file.  In normal circumstances it does, but in a set-UID or\n"
+           "set-GID program it doesn't because @code{access?} tests the\n"
+           "real ID, whereas an open or execute attempt uses the effective\n"
+           "ID.\n"
+           "\n"
+           "A program which will never run set-UID/GID can ignore the\n"
+           "difference between real and effective IDs, but for maximum\n"
+           "generality, especially in library functions, it's best not to\n"
+           "use @code{access?} to predict the result of an open or execute,\n"
+           "instead simply attempt that and catch any exception.\n"
+           "\n"
+           "The main use for @code{access?} is to let a set-UID/GID program\n"
+           "determine what the invoking user would have been allowed to do,\n"
+           "without the greater (or perhaps lesser) privileges afforded by\n"
+           "the effective ID.  For more on this, see ``Testing File\n"
+           "Access'' in The GNU C Library Reference Manual.")
 #define FUNC_NAME s_scm_access
 {
   int rv;
 
-  SCM_VALIDATE_STRING (1, path);
-  SCM_VALIDATE_INUM (2, how);
-  rv = access (SCM_STRING_CHARS (path), SCM_INUM (how));
-  return SCM_NEGATE_BOOL(rv);
+  WITH_STRING (path, c_path,
+              rv = access (c_path, scm_to_int (how)));
+  return scm_from_bool (!rv);
 }
 #undef FUNC_NAME
 
@@ -1220,7 +1293,7 @@ SCM_DEFINE (scm_getpid, "getpid", 0, 0, 0,
            "Return an integer representing the current process ID.")
 #define FUNC_NAME s_scm_getpid
 {
-  return SCM_MAKINUM ((unsigned long) getpid ());
+  return scm_from_ulong (getpid ());
 }
 #undef FUNC_NAME
 
@@ -1238,42 +1311,35 @@ SCM_DEFINE (scm_putenv, "putenv", 1, 0, 0,
 #define FUNC_NAME s_scm_putenv
 {
   int rv;
-  char *ptr;
-
-  SCM_VALIDATE_STRING (1, str);
+  char *c_str = scm_to_locale_string (str);
+#ifdef __MINGW32__
+  size_t len = strlen (c_str);
+#endif
 
-  if (strchr (SCM_STRING_CHARS (str), '=') == NULL)
+  if (strchr (c_str, '=') == NULL)
     {
 #ifdef HAVE_UNSETENV
       /* No '=' in argument means we should remove the variable from
         the environment.  Not all putenvs understand this (for instance
         FreeBSD 4.8 doesn't).  To be safe, we do it explicitely using
         unsetenv. */
-      unsetenv (SCM_STRING_CHARS (str));
+      unsetenv (c_str);
+      free (c_str);
 #else
       /* On e.g. Win32 hosts putenv() called with 'name=' removes the
         environment variable 'name'. */
       int e;
-      ptr = scm_malloc (SCM_STRING_LENGTH (str) + 2);
-      if (ptr == NULL)
-       SCM_MEMORY_ERROR;
-      strncpy (ptr, SCM_STRING_CHARS (str), SCM_STRING_LENGTH (str));
-      ptr[SCM_STRING_LENGTH (str)] = '=';
-      ptr[SCM_STRING_LENGTH (str) + 1] = 0;
+      char *ptr = scm_malloc (len + 2);
+      strcpy (ptr, c_str);
+      strcpy (ptr+len, "=");
       rv = putenv (ptr);
-      e = errno; free (ptr); errno = e;
+      e = errno; free (ptr); free (c_str); errno = e;
       if (rv < 0)
        SCM_SYSERROR;
 #endif /* !HAVE_UNSETENV */
     }
   else
     {
-      /* must make a new copy to be left in the environment, safe from gc.  */
-      ptr = scm_malloc (SCM_STRING_LENGTH (str) + 1);
-      if (ptr == NULL)
-       SCM_MEMORY_ERROR;
-      strncpy (ptr, SCM_STRING_CHARS (str), SCM_STRING_LENGTH (str));
-
 #ifdef __MINGW32__
       /* If str is "FOO=", ie. attempting to set an empty string, then
          we need to see if it's been successful.  On MINGW, "FOO="
@@ -1281,35 +1347,32 @@ SCM_DEFINE (scm_putenv, "putenv", 1, 0, 0,
          set "FOO= ", ie. a space, and then modify the string returned
          by getenv.  It's not enough just to modify the string we set,
          because MINGW putenv copies it.  */
-      if (ptr[SCM_STRING_LENGTH (str) - 1] == '=')
+
+      if (c_str[len-1] == '=')
         {
-         char *alt;
-          SCM name = scm_substring (str, SCM_MAKINUM (0),
-                                    SCM_MAKINUM (SCM_STRING_LENGTH (str) - 1));
-          if (getenv (SCM_STRING_CHARS (name)) == NULL)
-            {
-              alt = scm_malloc (SCM_STRING_LENGTH (str) + 2);
-             if (alt == NULL)
-               {
-                 free (ptr);
-                 SCM_MEMORY_ERROR;
-               }
-              memcpy (alt, SCM_STRING_CHARS (str), SCM_STRING_LENGTH (str));
-              alt[SCM_STRING_LENGTH (str)] = ' ';
-              alt[SCM_STRING_LENGTH (str) + 1] = '\0';
-              rv = putenv (alt);
-              if (rv < 0)
-               SCM_SYSERROR;
-              free (ptr);   /* don't need the old string we gave to putenv */
+         char *ptr = scm_malloc (len+2);
+         strcpy (ptr, c_str);
+         strcpy (ptr+len, " ");
+         rv = putenv (ptr);
+         if (rv < 0)
+           {
+             int eno = errno;
+             free (c_str);
+             errno = eno;
+             SCM_SYSERROR;
            }
-         alt = getenv (SCM_STRING_CHARS (name));
-         alt[0] = '\0';
+         /* truncate to just the name */
+         c_str[len-1] = '\0';
+         ptr = getenv (c_str);
+         if (ptr)
+           ptr[0] = '\0';
          return SCM_UNSPECIFIED;
         }
 #endif /* __MINGW32__ */
 
-      ptr[SCM_STRING_LENGTH (str)] = 0;
-      rv = putenv (ptr);
+      /* Leave c_str in the environment.  */
+
+      rv = putenv (c_str);
       if (rv < 0)
        SCM_SYSERROR;
     }
@@ -1334,21 +1397,34 @@ SCM_DEFINE (scm_setlocale, "setlocale", 1, 1, 0,
   char *clocale;
   char *rv;
 
-  SCM_VALIDATE_INUM (1, category);
+  scm_dynwind_begin (0);
+
   if (SCM_UNBNDP (locale))
     {
       clocale = NULL;
     }
   else
     {
-      SCM_VALIDATE_STRING (2, locale);
-      clocale = SCM_STRING_CHARS (locale);
+      clocale = scm_to_locale_string (locale);
+      scm_dynwind_free (clocale);
     }
 
-  rv = setlocale (SCM_INUM (category), clocale);
+  rv = setlocale (scm_i_to_lc_category (category, 1), clocale);
   if (rv == NULL)
-    SCM_SYSERROR;
-  return scm_makfrom0str (rv);
+    {
+      /* POSIX and C99 don't say anything about setlocale setting errno, so
+         force a sensible value here.  glibc leaves ENOENT, which would be
+         fine, but it's not a documented feature.  */
+      errno = EINVAL;
+      SCM_SYSERROR;
+    }
+
+  /* Recompute the standard SRFI-14 character sets in a locale-dependent
+     (actually charset-dependent) way.  */
+  scm_srfi_14_compute_char_sets ();
+
+  scm_dynwind_end ();
+  return scm_from_locale_string (rv);
 }
 #undef FUNC_NAME
 #endif /* HAVE_SETLOCALE */
@@ -1372,21 +1448,22 @@ SCM_DEFINE (scm_mknod, "mknod", 4, 0, 0,
 #define FUNC_NAME s_scm_mknod
 {
   int val;
-  char *p;
+  const char *p;
   int ctype = 0;
 
   SCM_VALIDATE_STRING (1, path);
   SCM_VALIDATE_SYMBOL (2, type);
-  SCM_VALIDATE_INUM (3, perms);
-  SCM_VALIDATE_INUM (4, dev);
 
-  p = SCM_SYMBOL_CHARS (type);
+  p = scm_i_symbol_chars (type);
   if (strcmp (p, "regular") == 0)
     ctype = S_IFREG;
   else if (strcmp (p, "directory") == 0)
     ctype = S_IFDIR;
+#ifdef S_IFLNK
+  /* systems without symlinks probably don't have S_IFLNK defined */
   else if (strcmp (p, "symlink") == 0)
     ctype = S_IFLNK;
+#endif
   else if (strcmp (p, "block-special") == 0)
     ctype = S_IFBLK;
   else if (strcmp (p, "char-special") == 0)
@@ -1400,8 +1477,10 @@ SCM_DEFINE (scm_mknod, "mknod", 4, 0, 0,
   else
     SCM_OUT_OF_RANGE (2, type);
 
-  SCM_SYSCALL (val = mknod (SCM_STRING_CHARS (path), ctype | SCM_INUM (perms),
-                           SCM_INUM (dev)));
+  STRING_SYSCALL (path, c_path,
+                 val = mknod (c_path,
+                              ctype | scm_to_int (perms),
+                              scm_to_int (dev)));
   if (val != 0)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
@@ -1417,8 +1496,11 @@ SCM_DEFINE (scm_nice, "nice", 1, 0, 0,
            "The return value is unspecified.")
 #define FUNC_NAME s_scm_nice
 {
-  SCM_VALIDATE_INUM (1, incr);
-  if (nice(SCM_INUM(incr)) != 0)
+  /* nice() returns "prio-NZERO" on success or -1 on error, but -1 can arise
+     from "prio-NZERO", so an error must be detected from errno changed */
+  errno = 0;
+  nice (scm_to_int (incr));
+  if (errno != 0)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
 }
@@ -1438,20 +1520,46 @@ SCM_DEFINE (scm_sync, "sync", 0, 0, 0,
 #undef FUNC_NAME
 #endif /* HAVE_SYNC */
 
+
+/* crypt() returns a pointer to a static buffer, so we use scm_i_misc_mutex
+   to avoid another thread overwriting it.  A test program running crypt
+   continuously in two threads can be quickly seen tripping this problem.
+   crypt() is pretty slow normally, so a mutex shouldn't add much overhead.
+
+   glibc has a thread-safe crypt_r, but (in version 2.3.2) it runs a lot
+   slower (about 5x) than plain crypt if you pass an uninitialized data
+   block each time.  Presumably there's some one-time setups.  The best way
+   to use crypt_r for parallel execution in multiple threads would probably
+   be to maintain a little pool of initialized crypt_data structures, take
+   one and use it, then return it to the pool.  That pool could be garbage
+   collected so it didn't add permanently to memory use if only a few crypt
+   calls are made.  But we expect crypt will be used rarely, and even more
+   rarely will there be any desire for lots of parallel execution on
+   multiple cpus.  So for now we don't bother with anything fancy, just
+   ensure it works.  */
+
 #if HAVE_CRYPT
-SCM_DEFINE (scm_crypt, "crypt", 2, 0, 0, 
+SCM_DEFINE (scm_crypt, "crypt", 2, 0, 0,
             (SCM key, SCM salt),
            "Encrypt @var{key} using @var{salt} as the salt value to the\n"
            "crypt(3) library call.")
 #define FUNC_NAME s_scm_crypt
 {
-  char * p;
+  SCM ret;
+  char *c_key, *c_salt;
+
+  scm_dynwind_begin (0);
+  scm_i_dynwind_pthread_mutex_lock (&scm_i_misc_mutex);
 
-  SCM_VALIDATE_STRING (1, key);
-  SCM_VALIDATE_STRING (2, salt);
+  c_key = scm_to_locale_string (key);
+  scm_dynwind_free (c_key);
+  c_salt = scm_to_locale_string (salt);
+  scm_dynwind_free (c_salt);
 
-  p = crypt (SCM_STRING_CHARS (key), SCM_STRING_CHARS (salt));
-  return scm_makfrom0str (p);
+  ret = scm_from_locale_string (crypt (c_key, c_salt));
+
+  scm_dynwind_end ();
+  return ret;
 }
 #undef FUNC_NAME
 #endif /* HAVE_CRYPT */
@@ -1466,9 +1574,11 @@ SCM_DEFINE (scm_chroot, "chroot", 1, 0, 0,
            "root directory.")
 #define FUNC_NAME s_scm_chroot
 {
-  SCM_VALIDATE_STRING (1, path);
+  int rv;
 
-  if (chroot (SCM_STRING_CHARS (path)) == -1)
+  WITH_STRING (path, c_path,
+              rv = chroot (c_path));
+  if (rv == -1)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
 }
@@ -1503,7 +1613,7 @@ SCM_DEFINE (scm_getlogin, "getlogin", 0, 0, 0,
   p = getlogin ();
   if (!p || !*p)
     return SCM_BOOL_F;
-  return scm_makfrom0str (p);
+  return scm_from_locale_string (p);
 }
 #undef FUNC_NAME
 #endif /* HAVE_GETLOGIN */
@@ -1522,7 +1632,7 @@ SCM_DEFINE (scm_cuserid, "cuserid", 0, 0, 0,
   p = cuserid (buf);
   if (!p || !*p)
     return SCM_BOOL_F;
-  return scm_makfrom0str (p);
+  return scm_from_locale_string (p);
 }
 #undef FUNC_NAME
 #endif /* HAVE_CUSERID */
@@ -1544,8 +1654,8 @@ SCM_DEFINE (scm_getpriority, "getpriority", 2, 0, 0,
 {
   int cwhich, cwho, ret;
 
-  SCM_VALIDATE_INUM_COPY (1, which, cwhich);
-  SCM_VALIDATE_INUM_COPY (2, who, cwho);
+  cwhich = scm_to_int (which);
+  cwho = scm_to_int (who);
 
   /* We have to clear errno and examine it later, because -1 is a
      legal return value for getpriority().  */
@@ -1553,7 +1663,7 @@ SCM_DEFINE (scm_getpriority, "getpriority", 2, 0, 0,
   ret = getpriority (cwhich, cwho);
   if (errno != 0)
     SCM_SYSERROR;
-  return SCM_MAKINUM (ret);
+  return scm_from_int (ret);
 }
 #undef FUNC_NAME
 #endif /* HAVE_GETPRIORITY */
@@ -1578,9 +1688,9 @@ SCM_DEFINE (scm_setpriority, "setpriority", 3, 0, 0,
 {
   int cwhich, cwho, cprio;
 
-  SCM_VALIDATE_INUM_COPY (1, which, cwhich);
-  SCM_VALIDATE_INUM_COPY (2, who, cwho);
-  SCM_VALIDATE_INUM_COPY (3, prio, cprio);
+  cwhich = scm_to_int (which);
+  cwho = scm_to_int (who);
+  cprio = scm_to_int (prio);
 
   if (setpriority (cwhich, cwho, cprio) == -1)
     SCM_SYSERROR;
@@ -1606,8 +1716,9 @@ SCM_DEFINE (scm_getpass, "getpass", 1, 0, 0,
 
   SCM_VALIDATE_STRING (1, prompt);
 
-  p = getpass(SCM_STRING_CHARS (prompt));
-  passwd = scm_makfrom0str (p);
+  WITH_STRING (prompt, c_prompt, 
+              p = getpass(c_prompt));
+  passwd = scm_from_locale_string (p);
 
   /* Clear out the password in the static buffer.  */
   memset (p, 0, strlen (p));
@@ -1688,35 +1799,41 @@ SCM_DEFINE (scm_flock, "flock", 2, 0, 0,
             (SCM file, SCM operation),
            "Apply or remove an advisory lock on an open file.\n"
            "@var{operation} specifies the action to be done:\n"
-           "@table @code\n"
-           "@item LOCK_SH\n"
+           "\n"
+           "@defvar LOCK_SH\n"
            "Shared lock.  More than one process may hold a shared lock\n"
            "for a given file at a given time.\n"
-           "@item LOCK_EX\n"
+           "@end defvar\n"
+           "@defvar LOCK_EX\n"
            "Exclusive lock.  Only one process may hold an exclusive lock\n"
            "for a given file at a given time.\n"
-           "@item LOCK_UN\n"
+           "@end defvar\n"
+           "@defvar LOCK_UN\n"
            "Unlock the file.\n"
-           "@item LOCK_NB\n"
-           "Don't block when locking.  May be specified by bitwise OR'ing\n"
-           "it to one of the other operations.\n"
-           "@end table\n"
+           "@end defvar\n"
+           "@defvar LOCK_NB\n"
+           "Don't block when locking.  This is combined with one of the\n"
+           "other operations using @code{logior}.  If @code{flock} would\n"
+           "block an @code{EWOULDBLOCK} error is thrown.\n"
+           "@end defvar\n"
+           "\n"
            "The return value is not specified. @var{file} may be an open\n"
-           "file descriptor or an open file descriptor port.")
+           "file descriptor or an open file descriptor port.\n"
+           "\n"
+           "Note that @code{flock} does not lock files across NFS.")
 #define FUNC_NAME s_scm_flock
 {
-  int coperation, fdes;
+  int fdes;
 
-  if (SCM_INUMP (file))
-    fdes = SCM_INUM (file);
+  if (scm_is_integer (file))
+    fdes = scm_to_int (file);
   else
     {
       SCM_VALIDATE_OPFPORT (2, file);
 
       fdes = SCM_FPORT_FDES (file);
     }
-  SCM_VALIDATE_INUM_COPY (2, operation, coperation);
-  if (flock (fdes, coperation) == -1)
+  if (flock (fdes, scm_to_int (operation)) == -1)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
 }
@@ -1731,46 +1848,101 @@ SCM_DEFINE (scm_sethostname, "sethostname", 1, 0, 0,
            "specified.")
 #define FUNC_NAME s_scm_sethostname
 {
-  SCM_VALIDATE_STRING (1, name);
+  int rv;
 
-  if (sethostname (SCM_STRING_CHARS (name), SCM_STRING_LENGTH (name)) == -1)
+  WITH_STRING (name, c_name,
+              rv = sethostname (c_name, strlen(c_name)));
+  if (rv == -1)
     SCM_SYSERROR;
   return SCM_UNSPECIFIED;
 }
 #undef FUNC_NAME
 #endif /* HAVE_SETHOSTNAME */
 
+
 #if HAVE_GETHOSTNAME
 SCM_DEFINE (scm_gethostname, "gethostname", 0, 0, 0, 
             (void),
            "Return the host name of the current processor.")
 #define FUNC_NAME s_scm_gethostname
 {
-  /* 256 is for Solaris, under Linux ENAMETOOLONG is returned if not
-     large enough.  */
-  int len = 256, res;
-  char *p = scm_malloc (len);
-  SCM name;
+#ifdef MAXHOSTNAMELEN
+
+  /* Various systems define MAXHOSTNAMELEN (including Solaris in fact).
+   * On GNU/Linux this doesn't include the terminating '\0', hence "+ 1".  */
+  const int len = MAXHOSTNAMELEN + 1;
+  char *const p = scm_malloc (len);
+  const int res = gethostname (p, len);
+
+  scm_dynwind_begin (0);
+  scm_dynwind_unwind_handler (free, p, 0);
+
+#else
+
+  /* Default 256 is for Solaris, under Linux ENAMETOOLONG is returned if not
+   * large enough.  SUSv2 specifies 255 maximum too, apparently.  */
+  int len = 256;
+  int res;
+  char *p;
+
+#  if HAVE_SYSCONF && defined (_SC_HOST_NAME_MAX)
+
+  /* POSIX specifies the HOST_NAME_MAX system parameter for the max size,
+   * which may reflect a particular kernel configuration.
+   * Must watch out for this existing but giving -1, as happens for instance
+   * in gnu/linux glibc 2.3.2.  */
+  {
+    const long int n = sysconf (_SC_HOST_NAME_MAX);
+    if (n != -1L)
+      len = n;
+  }
+
+#  endif
+
+  p = scm_malloc (len);
+
+  scm_dynwind_begin (0);
+  scm_dynwind_unwind_handler (free, p, 0);
 
   res = gethostname (p, len);
   while (res == -1 && errno == ENAMETOOLONG)
     {
-      p = scm_realloc (p, len * 2);
       len *= 2;
+
+      /* scm_realloc may throw an exception.  */
+      p = scm_realloc (p, len);
       res = gethostname (p, len);
     }
+
+#endif
+
   if (res == -1)
     {
+      const int save_errno = errno;
+
+      /* No guile exceptions can occur before we have freed p's memory. */
+      scm_dynwind_end ();
       free (p);
+
+      errno = save_errno;
       SCM_SYSERROR;
     }
-  name = scm_makfrom0str (p);
-  free (p);
-  return name;
+  else
+    {
+      /* scm_from_locale_string may throw an exception.  */
+      const SCM name = scm_from_locale_string (p);
+
+      /* No guile exceptions can occur before we have freed p's memory. */
+      scm_dynwind_end ();
+      free (p);
+
+      return name;
+    }
 }
 #undef FUNC_NAME
 #endif /* HAVE_GETHOSTNAME */
 
+
 void 
 scm_init_posix ()
 {
@@ -1779,70 +1951,88 @@ scm_init_posix ()
   scm_add_feature ("EIDs");
 #endif
 #ifdef WAIT_ANY
-  scm_c_define ("WAIT_ANY", SCM_MAKINUM (WAIT_ANY));
+  scm_c_define ("WAIT_ANY", scm_from_int (WAIT_ANY));
 #endif
 #ifdef WAIT_MYPGRP
-  scm_c_define ("WAIT_MYPGRP", SCM_MAKINUM (WAIT_MYPGRP));
+  scm_c_define ("WAIT_MYPGRP", scm_from_int (WAIT_MYPGRP));
 #endif
 #ifdef WNOHANG
-  scm_c_define ("WNOHANG", SCM_MAKINUM (WNOHANG));
+  scm_c_define ("WNOHANG", scm_from_int (WNOHANG));
 #endif
 #ifdef WUNTRACED
-  scm_c_define ("WUNTRACED", SCM_MAKINUM (WUNTRACED));
+  scm_c_define ("WUNTRACED", scm_from_int (WUNTRACED));
 #endif
 
   /* access() symbols.  */
-  scm_c_define ("R_OK", SCM_MAKINUM (R_OK));
-  scm_c_define ("W_OK", SCM_MAKINUM (W_OK));
-  scm_c_define ("X_OK", SCM_MAKINUM (X_OK));
-  scm_c_define ("F_OK", SCM_MAKINUM (F_OK));
+  scm_c_define ("R_OK", scm_from_int (R_OK));
+  scm_c_define ("W_OK", scm_from_int (W_OK));
+  scm_c_define ("X_OK", scm_from_int (X_OK));
+  scm_c_define ("F_OK", scm_from_int (F_OK));
 
 #ifdef LC_COLLATE
-  scm_c_define ("LC_COLLATE", SCM_MAKINUM (LC_COLLATE));
+  scm_c_define ("LC_COLLATE", scm_from_int (LC_COLLATE));
 #endif
 #ifdef LC_CTYPE
-  scm_c_define ("LC_CTYPE", SCM_MAKINUM (LC_CTYPE));
+  scm_c_define ("LC_CTYPE", scm_from_int (LC_CTYPE));
 #endif
 #ifdef LC_MONETARY
-  scm_c_define ("LC_MONETARY", SCM_MAKINUM (LC_MONETARY));
+  scm_c_define ("LC_MONETARY", scm_from_int (LC_MONETARY));
 #endif
 #ifdef LC_NUMERIC
-  scm_c_define ("LC_NUMERIC", SCM_MAKINUM (LC_NUMERIC));
+  scm_c_define ("LC_NUMERIC", scm_from_int (LC_NUMERIC));
 #endif
 #ifdef LC_TIME
-  scm_c_define ("LC_TIME", SCM_MAKINUM (LC_TIME));
+  scm_c_define ("LC_TIME", scm_from_int (LC_TIME));
 #endif
 #ifdef LC_MESSAGES
-  scm_c_define ("LC_MESSAGES", SCM_MAKINUM (LC_MESSAGES));
+  scm_c_define ("LC_MESSAGES", scm_from_int (LC_MESSAGES));
 #endif
 #ifdef LC_ALL
-  scm_c_define ("LC_ALL", SCM_MAKINUM (LC_ALL));
+  scm_c_define ("LC_ALL", scm_from_int (LC_ALL));
+#endif
+#ifdef LC_PAPER
+  scm_c_define ("LC_PAPER", scm_from_int (LC_PAPER));
+#endif
+#ifdef LC_NAME
+  scm_c_define ("LC_NAME", scm_from_int (LC_NAME));
+#endif
+#ifdef LC_ADDRESS
+  scm_c_define ("LC_ADDRESS", scm_from_int (LC_ADDRESS));
+#endif
+#ifdef LC_TELEPHONE
+  scm_c_define ("LC_TELEPHONE", scm_from_int (LC_TELEPHONE));
+#endif
+#ifdef LC_MEASUREMENT
+  scm_c_define ("LC_MEASUREMENT", scm_from_int (LC_MEASUREMENT));
+#endif
+#ifdef LC_IDENTIFICATION
+  scm_c_define ("LC_IDENTIFICATION", scm_from_int (LC_IDENTIFICATION));
 #endif
 #ifdef PIPE_BUF
-  scm_c_define ("PIPE_BUF", scm_long2num (PIPE_BUF));
+  scm_c_define ("PIPE_BUF", scm_from_long (PIPE_BUF));
 #endif
 
 #ifdef PRIO_PROCESS
-  scm_c_define ("PRIO_PROCESS", SCM_MAKINUM (PRIO_PROCESS));
+  scm_c_define ("PRIO_PROCESS", scm_from_int (PRIO_PROCESS));
 #endif
 #ifdef PRIO_PGRP
-  scm_c_define ("PRIO_PGRP", SCM_MAKINUM (PRIO_PGRP));
+  scm_c_define ("PRIO_PGRP", scm_from_int (PRIO_PGRP));
 #endif
 #ifdef PRIO_USER
-  scm_c_define ("PRIO_USER", SCM_MAKINUM (PRIO_USER));
+  scm_c_define ("PRIO_USER", scm_from_int (PRIO_USER));
 #endif
 
 #ifdef LOCK_SH
-  scm_c_define ("LOCK_SH", SCM_MAKINUM (LOCK_SH));
+  scm_c_define ("LOCK_SH", scm_from_int (LOCK_SH));
 #endif
 #ifdef LOCK_EX
-  scm_c_define ("LOCK_EX", SCM_MAKINUM (LOCK_EX));
+  scm_c_define ("LOCK_EX", scm_from_int (LOCK_EX));
 #endif
 #ifdef LOCK_UN
-  scm_c_define ("LOCK_UN", SCM_MAKINUM (LOCK_UN));
+  scm_c_define ("LOCK_UN", scm_from_int (LOCK_UN));
 #endif
 #ifdef LOCK_NB
-  scm_c_define ("LOCK_NB", SCM_MAKINUM (LOCK_NB));
+  scm_c_define ("LOCK_NB", scm_from_int (LOCK_NB));
 #endif
 
 #include "libguile/cpp_sig_symbols.c"