errno: use consistently

* lib/c-stack.c (ENOTSUP): <errno.h> guarantees a definition.
* lib/canonicalize-lgpl.c (ENAMETOOLONG): Likewise.
* lib/canonicalize.c (ELOOP): Likewise.
* lib/inet_ntop.c (EAFNOSUPPORT): Likewise.
* lib/inet_pton.c (EAFNOSUPPORT): Likewise.
* lib/lchown.c (EOPNOTSUPP): Likewise.
* lib/openat-priv.h (ENOSYS, EOPNOTSUPP): Likewise.
* lib/savewd.c (ESTALE): Likewise.
* lib/settime.c (ENOSYS): Likewise.
* lib/utimens.c (ENOSYS): Likewise.
* lib/xgethostname.c (ENAMETOOLONG): Likewise.
* lib/chdir-safer.c (ELOOP): Likewise.
(chdir_no_follow): Use HAVE_READLINK, not ELOOP, as witness.
* modules/c-stack (Depends-on): Add errno.
* modules/canonicalize (Depends-on): Likewise.
* modules/chdir-safer (Depends-on): Likewise.
* modules/fdopendir (Depends-on): Likewise.
* modules/inet_ntop (Depends-on): Likewise.
* modules/inet_pton (Depends-on): Likewise.
* modules/lchown (Depends-on): Likewise.
* modules/openat (Depends-on): Likewise.
* modules/savewd (Depends-on): Likewise.
* modules/settime (Depends-on): Likewise.
* m4/chdir-safer.m4 (gl_CHDIR_SAFER): Check for readlink.

Signed-off-by: Eric Blake <ebb9@byu.net>

diff --git a/ChangeLog b/ChangeLog
index 1f8d5e80010ce01ce2225f68f40df879f199ac39..4eeb067c790a0e32c89e22638f7d695dd2edbde8 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,31 @@
 2009-09-02  Eric Blake  <ebb9@byu.net>
 
+       errno: use consistently
+       * lib/c-stack.c (ENOTSUP): <errno.h> guarantees a definition.
+       * lib/canonicalize-lgpl.c (ENAMETOOLONG): Likewise.
+       * lib/canonicalize.c (ELOOP): Likewise.
+       * lib/inet_ntop.c (EAFNOSUPPORT): Likewise.
+       * lib/inet_pton.c (EAFNOSUPPORT): Likewise.
+       * lib/lchown.c (EOPNOTSUPP): Likewise.
+       * lib/openat-priv.h (ENOSYS, EOPNOTSUPP): Likewise.
+       * lib/savewd.c (ESTALE): Likewise.
+       * lib/settime.c (ENOSYS): Likewise.
+       * lib/utimens.c (ENOSYS): Likewise.
+       * lib/xgethostname.c (ENAMETOOLONG): Likewise.
+       * lib/chdir-safer.c (ELOOP): Likewise.
+       (chdir_no_follow): Use HAVE_READLINK, not ELOOP, as witness.
+       * modules/c-stack (Depends-on): Add errno.
+       * modules/canonicalize (Depends-on): Likewise.
+       * modules/chdir-safer (Depends-on): Likewise.
+       * modules/fdopendir (Depends-on): Likewise.
+       * modules/inet_ntop (Depends-on): Likewise.
+       * modules/inet_pton (Depends-on): Likewise.
+       * modules/lchown (Depends-on): Likewise.
+       * modules/openat (Depends-on): Likewise.
+       * modules/savewd (Depends-on): Likewise.
+       * modules/settime (Depends-on): Likewise.
+       * m4/chdir-safer.m4 (gl_CHDIR_SAFER): Check for readlink.
+
        fts: avoid leaking fds
        * modules/fts (Depends-on): Add cloexec.
        * lib/fts.c (opendirat, diropen, fts_build): Set close-on-exec

diff --git a/lib/c-stack.c b/lib/c-stack.c
index d260b47aa546122340ed91ba9e3e4e897c485b4c..1dddeefc412d1242c03839bebb583b1fbfc6dcf3 100644 (file)
--- a/lib/c-stack.c
+++ b/lib/c-stack.c
@@ -46,9 +46,6 @@
 #define _(msgid) gettext (msgid)
 
 #include <errno.h>
-#ifndef ENOTSUP
-# define ENOTSUP EINVAL
-#endif
 
 #include <signal.h>
 #if ! HAVE_STACK_T && ! defined stack_t

diff --git a/lib/canonicalize-lgpl.c b/lib/canonicalize-lgpl.c
index 3aaa310ecd2d0883e6b8e5121e214859d28d1786..6b5663a3deadedd1c2c954004e550e032cff97ce 100644 (file)
--- a/lib/canonicalize-lgpl.c
+++ b/lib/canonicalize-lgpl.c
@@ -1,5 +1,5 @@
 /* Return the canonical absolute name of a given file.
-   Copyright (C) 1996-2003, 2005-2008 Free Software Foundation, Inc.
+   Copyright (C) 1996-2003, 2005-2009 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    This program is free software: you can redistribute it and/or modify
@@ -50,9 +50,6 @@
 #include <errno.h>
 #ifndef _LIBC
 # define __set_errno(e) errno = (e)
-# ifndef ENAMETOOLONG
-#  define ENAMETOOLONG EINVAL
-# endif
 #endif
 
 #ifdef _LIBC

diff --git a/lib/canonicalize.c b/lib/canonicalize.c
index 37446545cdfe21b1a3682a15f9329f9b1f17564c..523e08204cf0320695b6740e34b95551262163ce 100644 (file)
--- a/lib/canonicalize.c
+++ b/lib/canonicalize.c
@@ -38,9 +38,6 @@
 #include "xalloc.h"
 #include "xgetcwd.h"
 
-#ifndef ELOOP
-# define ELOOP 0
-#endif
 #ifndef __set_errno
 # define __set_errno(Val) errno = (Val)
 #endif

diff --git a/lib/chdir-safer.c b/lib/chdir-safer.c
index 03f372050f9decc43e326e406d7f9aa3af0b5821..a6f77b0afc8c3e5b772248a7126a44a1805605cc 100644 (file)
--- a/lib/chdir-safer.c
+++ b/lib/chdir-safer.c
@@ -1,6 +1,6 @@
 /* much like chdir(2), but safer
 
-   Copyright (C) 2005-2006, 2008 Free Software Foundation, Inc.
+   Copyright (C) 2005-2006, 2008-2009 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
@@ -29,8 +29,8 @@
 #include <sys/stat.h>
 #include "same-inode.h"
 
-#ifndef ELOOP
-# define ELOOP 0
+#ifndef HAVE_READLINK
+# define HAVE_READLINK 0
 #endif
 
 /* Like chdir, but fail if DIR is a symbolic link to a directory (or
@@ -51,9 +51,9 @@ chdir_no_follow (char const *dir)
      they are the same file; if they are different files, set errno to
      ELOOP (the same value that open uses for symlinks with
      O_NOFOLLOW) so the caller can report a failure.
-     Skip this check if ELOOP == 0, which should be the case
+     Skip this check if HAVE_READLINK == 0, which should be the case
      on any system that lacks symlink support.  */
-  if (ELOOP && ! HAVE_WORKING_O_NOFOLLOW)
+  if (HAVE_READLINK && ! HAVE_WORKING_O_NOFOLLOW)
     {
       struct stat sb1;
       result = lstat (dir, &sb1);

diff --git a/lib/inet_ntop.c b/lib/inet_ntop.c
index 16eb109210e2e59658073c9576096eb9eb8d659a..033847a94bd9b645bbcd1838864d0f022ead8e2a 100644 (file)
--- a/lib/inet_ntop.c
+++ b/lib/inet_ntop.c
@@ -1,6 +1,6 @@
 /* inet_ntop.c -- convert IPv4 and IPv6 addresses from binary to text form
 
-   Copyright (C) 2005, 2006, 2008  Free Software Foundation, Inc.
+   Copyright (C) 2005, 2006, 2008, 2009  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
@@ -42,10 +42,6 @@
 #include <string.h>
 #include <errno.h>
 
-#ifndef EAFNOSUPPORT
-# define EAFNOSUPPORT EINVAL
-#endif
-
 #define NS_IN6ADDRSZ 16
 #define NS_INT16SZ 2
 

diff --git a/lib/inet_pton.c b/lib/inet_pton.c
index 62ef145950544800c84e1bf468997a07ada12404..a9c9bafdef18fc307bc4c6fe71a3ab49c8427cd3 100644 (file)
--- a/lib/inet_pton.c
+++ b/lib/inet_pton.c
@@ -1,6 +1,6 @@
 /* inet_pton.c -- convert IPv4 and IPv6 addresses from text to binary form
 
-   Copyright (C) 2006, 2008 Free Software Foundation, Inc.
+   Copyright (C) 2006, 2008, 2009 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
@@ -41,10 +41,6 @@
 #include <string.h>
 #include <errno.h>
 
-#ifndef EAFNOSUPPORT
-# define EAFNOSUPPORT EINVAL
-#endif
-
 #define NS_INADDRSZ 4
 #define NS_IN6ADDRSZ 16
 #define NS_INT16SZ 2

diff --git a/lib/lchown.c b/lib/lchown.c
index 5bc7074f819cf56e6763d3ad106946c65beb4e1c..10ba1c9e5c0d1899852b876932e40d0d662731c7 100644 (file)
--- a/lib/lchown.c
+++ b/lib/lchown.c
@@ -30,16 +30,6 @@
 #include <errno.h>
 #include <sys/stat.h>
 
-/* Some systems don't have EOPNOTSUPP.  */
-#ifndef EOPNOTSUPP
-# ifdef ENOTSUP
-#  define EOPNOTSUPP ENOTSUP
-# else
-/* Some systems don't have ENOTSUP either.  */
-#  define EOPNOTSUPP EINVAL
-# endif
-#endif
-
 /* Work just like chown, except when FILE is a symbolic link.
    In that case, set errno to EOPNOTSUPP and return -1.
    But if autoconf tests determined that chown modifies

diff --git a/lib/openat-priv.h b/lib/openat-priv.h
index 9254f6bbad85cf37ca17770e91c422abd0f3210d..2280416292d29075914ca15ab20e8e202a7ee197 100644 (file)
--- a/lib/openat-priv.h
+++ b/lib/openat-priv.h
@@ -1,6 +1,6 @@
 /* Internals for openat-like functions.
 
-   Copyright (C) 2005, 2006 Free Software Foundation, Inc.
+   Copyright (C) 2005, 2006, 2009 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
@@ -23,26 +23,6 @@
 #define OPENAT_BUFFER_SIZE 512
 char *openat_proc_name (char buf[OPENAT_BUFFER_SIZE], int fd, char const *file);
 
-/* Some systems don't have ENOSYS.  */
-#ifndef ENOSYS
-# ifdef ENOTSUP
-#  define ENOSYS ENOTSUP
-# else
-/* Some systems don't have ENOTSUP either.  */
-#  define ENOSYS EINVAL
-# endif
-#endif
-
-/* Some systems don't have EOPNOTSUPP.  */
-#ifndef EOPNOTSUPP
-# ifdef ENOTSUP
-#  define EOPNOTSUPP ENOTSUP
-# else
-/* Some systems don't have ENOTSUP either.  */
-#  define EOPNOTSUPP EINVAL
-# endif
-#endif
-
 /* Trying to access a BUILD_PROC_NAME file will fail on systems without
    /proc support, and even on systems *with* ProcFS support.  Return
    nonzero if the failure may be legitimate, e.g., because /proc is not

diff --git a/lib/savewd.c b/lib/savewd.c
index c4893ef6f1f877b75cae747a8222f5f21ac1a31e..f593f94677b4c91e31f1bebee51198e75cdceb6e 100644 (file)
--- a/lib/savewd.c
+++ b/lib/savewd.c
@@ -1,6 +1,6 @@
 /* Save and restore the working directory, possibly using a child process.
 
-   Copyright (C) 2006, 2007 Free Software Foundation, Inc.
+   Copyright (C) 2006, 2007, 2009 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
@@ -34,10 +34,6 @@
 #include "dirname.h"
 #include "fcntl-safer.h"
 
-#ifndef ESTALE
-# define ESTALE -1
-#endif
-
 /* Save the working directory into *WD, if it hasn't been saved
    already.  Return true if a child has been forked to do the real
    work.  */

diff --git a/lib/settime.c b/lib/settime.c
index f565492681a756f211153e30f3a645a079c60c44..366a298d045486af345f01480bd090d1a7bc354d 100644 (file)
--- a/lib/settime.c
+++ b/lib/settime.c
@@ -1,6 +1,6 @@
 /* settime -- set the system clock
 
-   Copyright (C) 2002, 2004, 2005, 2006, 2007 Free Software
+   Copyright (C) 2002, 2004, 2005, 2006, 2007, 2009 Free Software
    Foundation, Inc.
 
    This program is free software: you can redistribute it and/or modify
@@ -27,16 +27,6 @@
 
 #include <errno.h>
 
-/* Some systems don't have ENOSYS.  */
-#ifndef ENOSYS
-# ifdef ENOTSUP
-#  define ENOSYS ENOTSUP
-# else
-/* Some systems don't have ENOTSUP either.  */
-#  define ENOSYS EINVAL
-# endif
-#endif
-
 /* Set the system time.  */
 
 int

diff --git a/lib/utimens.c b/lib/utimens.c
index 708de10989d1e703250a841d397cb2b736c5c2f4..f7bc75e65373cff1b7db7b3797f54253c4f81a23 100644 (file)
--- a/lib/utimens.c
+++ b/lib/utimens.c
@@ -1,7 +1,7 @@
 /* Set file access and modification times.
 
-   Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Free Software
-   Foundation, Inc.
+   Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009 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
@@ -44,16 +44,6 @@ struct utimbuf
 };
 #endif
 
-/* Some systems don't have ENOSYS.  */
-#ifndef ENOSYS
-# ifdef ENOTSUP
-#  define ENOSYS ENOTSUP
-# else
-/* Some systems don't have ENOTSUP either.  */
-#  define ENOSYS EINVAL
-# endif
-#endif
-
 #ifndef __attribute__
 # if __GNUC__ < 2 || (__GNUC__ == 2 && __GNUC_MINOR__ < 8)
 #  define __attribute__(x)

diff --git a/lib/xgethostname.c b/lib/xgethostname.c
index 875d49761832482dcc7b2dfc1afbe067fc71f4ff..d649b8dcffbd6678906d384e8e43522ab1046462 100644 (file)
--- a/lib/xgethostname.c
+++ b/lib/xgethostname.c
@@ -1,7 +1,7 @@
 /* xgethostname.c -- return current hostname with unlimited length
 
-   Copyright (C) 1992, 1996, 2000, 2001, 2003, 2004, 2005, 2006 Free
-   Software Foundation, Inc.
+   Copyright (C) 1992, 1996, 2000, 2001, 2003, 2004, 2005, 2006, 2009
+   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
@@ -29,10 +29,6 @@
 
 #include "xalloc.h"
 
-#ifndef ENAMETOOLONG
-# define ENAMETOOLONG 0
-#endif
-
 #ifndef INITIAL_HOSTNAME_LENGTH
 # define INITIAL_HOSTNAME_LENGTH 34
 #endif

diff --git a/m4/chdir-safer.m4 b/m4/chdir-safer.m4
index 22ad50673cfd3672464f8150b092e5aad9156776..d6e741d9dd42e4c6d1c7805709d62f576cdd8a19 100644 (file)
--- a/m4/chdir-safer.m4
+++ b/m4/chdir-safer.m4
@@ -1,5 +1,5 @@
-#serial 3
-dnl Copyright (C) 2005, 2006 Free Software Foundation, Inc.
+#serial 4
+dnl Copyright (C) 2005, 2006, 2009 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
 dnl with or without modifications, as long as this notice is preserved.
@@ -7,4 +7,5 @@ dnl with or without modifications, as long as this notice is preserved.
 AC_DEFUN([gl_CHDIR_SAFER],
 [
   AC_LIBOBJ([chdir-safer])
+  AC_CHECK_FUNCS_ONCE([readlink])
 ])

diff --git a/modules/c-stack b/modules/c-stack
index 6a589c3145c65607c8a6a97eab77def32dea6863..22dd18c1aff484e0e090de1eb1f5868a09872e57 100644 (file)
--- a/modules/c-stack
+++ b/modules/c-stack
@@ -8,6 +8,7 @@ m4/c-stack.m4
 
 Depends-on:
 gettext-h
+errno
 exitfail
 unistd
 raise

diff --git a/modules/canonicalize b/modules/canonicalize
index 23baa7594c6265be12eb68baeeef08d7f2593404..e24be3d1773416d9cc0226890da86d71d5de38ba 100644 (file)
--- a/modules/canonicalize
+++ b/modules/canonicalize
@@ -9,6 +9,7 @@ m4/canonicalize.m4
 
 Depends-on:
 areadlink-with-size
+errno
 file-set
 filenamecat
 hash-triple

diff --git a/modules/chdir-safer b/modules/chdir-safer
index b3b47e16f6aad84cce0ae9aa989ae054e770d459..39cfdc7f6b161f611a7f82150a3263d52ec34542 100644 (file)
--- a/modules/chdir-safer
+++ b/modules/chdir-safer
@@ -7,6 +7,7 @@ lib/chdir-safer.c
 m4/chdir-safer.m4
 
 Depends-on:
+errno
 fchdir
 fcntl-h
 open

diff --git a/modules/fdopendir b/modules/fdopendir
index 7a90aa367ff1496cc976499e31cf3a2776e8d831..83c2af7d3247b0902f88358f415859639987c2f7 100644 (file)
--- a/modules/fdopendir
+++ b/modules/fdopendir
@@ -8,8 +8,9 @@ lib/openat-proc.c
 m4/fdopendir.m4
 
 Depends-on:
-extensions
 dirent
+errno
+extensions
 fchdir
 openat-die
 save-cwd

diff --git a/modules/inet_ntop b/modules/inet_ntop
index 98df38e51c1df101ff4d40146bc3fead86335d6f..76ff39da7972f806de5830b4193123a2cbb4d70b 100644 (file)
--- a/modules/inet_ntop
+++ b/modules/inet_ntop
@@ -9,6 +9,7 @@ Depends-on:
 socklen
 sys_socket
 arpa_inet
+errno
 netinet_in
 extensions
 

diff --git a/modules/inet_pton b/modules/inet_pton
index 173710ab8181c4177c7b552e926f1bf4d37f9165..314bf0f9c2448d02b8a17d16c879a216dc45b7b1 100644 (file)
--- a/modules/inet_pton
+++ b/modules/inet_pton
@@ -10,6 +10,7 @@ c-ctype
 socklen
 sys_socket
 arpa_inet
+errno
 netinet_in
 extensions
 

diff --git a/modules/lchown b/modules/lchown
index e011e2bddbd9ef2bf7cce82e68a2a442370cb7ed..65c084c9333201a93cca4afc90e92a753f94d79a 100644 (file)
--- a/modules/lchown
+++ b/modules/lchown
@@ -7,6 +7,7 @@ m4/lchown.m4
 
 Depends-on:
 chown
+errno
 sys_stat
 unistd
 

diff --git a/modules/openat b/modules/openat
index 5c326a07d50a7076a056dc652da282465ff96fff..3ea5b4d7cc0a4e578fee188f5e252b6a6ca6158c 100644 (file)
--- a/modules/openat
+++ b/modules/openat
@@ -16,6 +16,7 @@ m4/mode_t.m4
 
 Depends-on:
 dirname
+errno
 extensions
 fchdir
 fdopendir

diff --git a/modules/savewd b/modules/savewd
index 360d4b9a389553e440a1e41660db781249a88a5a..6d655c17e37b557c6cf1830943be5252dc933a8c 100644 (file)
--- a/modules/savewd
+++ b/modules/savewd
@@ -8,6 +8,7 @@ m4/savewd.m4
 
 Depends-on:
 dirname
+errno
 exit
 fchdir
 fcntl-safer

diff --git a/modules/settime b/modules/settime
index 9ca84a027ee8ab4036b75d9b7f97282781761e0f..f88318ed89eb80fdaf913e86d0a4b7a04bc9dbe3 100644 (file)
--- a/modules/settime
+++ b/modules/settime
@@ -7,6 +7,7 @@ m4/settime.m4
 
 Depends-on:
 clock-time
+errno
 extensions
 sys_time
 timespec

diff --git a/modules/utimens b/modules/utimens
index caf8805d6a720d1f146d746b8f8b7227d66ac7af..425148521cd5c9cb30eb64b4623f3e95f42183cf 100644 (file)
--- a/modules/utimens
+++ b/modules/utimens
@@ -9,6 +9,7 @@ m4/utimens.m4
 m4/utimes.m4
 
 Depends-on:
+errno
 sys_time
 time