openat: fix GNU/Hurd bug in unlinkat

unlinkat(fd,"file/",0) mistakenly succeeded.

* m4/openat.m4 (gl_FUNC_OPENAT): Replace unlinkat if unlink is
broken.
* doc/posix-functions/unlink.texi (unlink): Document this.
* doc/posix-functions/unlinkat.texi (unlinkat): Likewise.

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

diff --git a/ChangeLog b/ChangeLog
index 7ab913532c1ff600eb56d365ebc2992e9cd79957..05a47306710aaf1ebadde95f04fc77d50e08b528 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2009-10-06  Eric Blake  <ebb9@byu.net>
 
+       openat: fix GNU/Hurd bug in unlinkat
+       * m4/openat.m4 (gl_FUNC_OPENAT): Replace unlinkat if unlink is
+       broken.
+       * doc/posix-functions/unlink.texi (unlink): Document this.
+       * doc/posix-functions/unlinkat.texi (unlinkat): Likewise.
+
        fdopendir: fix GNU/Hurd bug
        * m4/fdopendir.m4 (gl_FUNC_FDOPENDIR): Check for Hurd bug in
        allowing non-directory fds.

diff --git a/doc/posix-functions/unlink.texi b/doc/posix-functions/unlink.texi
index 1249c42861655ee94cea10ece9f46402066a5c05..1df1e30b365d426a50fd69c3780e53dbb8138900 100644 (file)
--- a/doc/posix-functions/unlink.texi
+++ b/doc/posix-functions/unlink.texi
@@ -10,7 +10,7 @@ Portability problems fixed by Gnulib:
 @itemize
 @item
 Some systems mistakenly succeed on @code{unlink("file/")}:
-Solaris 9.
+GNU/Hurd, Solaris 9.
 @end itemize
 
 Portability problems not fixed by Gnulib:

diff --git a/doc/posix-functions/unlinkat.texi b/doc/posix-functions/unlinkat.texi
index dfff9b9bc972c04e21b9109119ca28696a8ad594..1ec724e4ed73d42260553a482dc1861bbc550347 100644 (file)
--- a/doc/posix-functions/unlinkat.texi
+++ b/doc/posix-functions/unlinkat.texi
@@ -15,7 +15,7 @@ glibc 2.3.6, MacOS X 10.3, FreeBSD 6.0, NetBSD 3.0, OpenBSD 3.8, AIX
 But the replacement function is not safe to be used in libraries and is not multithread-safe.
 @item
 Some systems mistakenly succeed on @code{unlinkat(fd,"file/",flag)}:
-Solaris 9.
+GNU/Hurd, Solaris 9.
 @end itemize
 
 Portability problems not fixed by Gnulib:

diff --git a/m4/openat.m4 b/m4/openat.m4
index 42df3ee72d90df2b317a402d29ce8ebe35e2e7e3..6b4f95c76f468a8582b5ebf113f04d7ef45f3799 100644 (file)
--- a/m4/openat.m4
+++ b/m4/openat.m4
@@ -1,4 +1,4 @@
-# serial 24
+# serial 25
 # See if we need to use our replacement for Solaris' openat et al functions.
 
 dnl Copyright (C) 2004-2009 Free Software Foundation, Inc.
@@ -27,8 +27,14 @@ AC_DEFUN([gl_FUNC_OPENAT],
   AC_CHECK_FUNCS_ONCE([lchmod])
   AC_REPLACE_FUNCS([fchmodat fstatat mkdirat openat unlinkat])
   AC_REQUIRE([AC_FUNC_LSTAT_FOLLOWS_SLASHED_SYMLINK])
+  AC_REQUIRE([gl_FUNC_UNLINK])
   case $ac_cv_func_openat+$ac_cv_func_lstat_dereferences_slashed_symlink in
-  yes+yes) ;;
+  yes+yes)
+    # GNU/Hurd has unlinkat, but it has the same bug as unlink.
+    if test $REPLACE_UNLINK = 1; then
+      AC_LIBOBJ([unlinkat])
+      REPLACE_UNLINKAT=1
+    fi ;;
   yes+*)
     # Solaris 9 has *at functions, but uniformly mishandles trailing
     # slash in all of them.