changeset 33207:5a09cdf7d19a

fchownat: Replace also when chown has the trailing slash bug.
author Bruno Haible <bruno@clisp.org>
date Sat, 31 Jul 2010 16:01:58 +0200
parents ef3b25cbf61f
children 8d5e9ad42e77
files ChangeLog m4/openat.m4
diffstat 2 files changed, 20 insertions(+), 9 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Sat Jul 31 15:53:21 2010 +0200
+++ b/ChangeLog	Sat Jul 31 16:01:58 2010 +0200
@@ -1,3 +1,11 @@
+2010-07-31  Bruno Haible  <bruno@clisp.org>
+
+	fchownat: Replace also when chown has the trailing slash bug.
+	* m4/openat.m4 (gl_FUNC_FCHOWNAT): Move the test of REPLACE_CHOWN
+	outside the gl_FUNC_FCHOWNAT_DEREF_BUG invocation. Fixes regression
+	introduced on 2010-04-10.
+	Reported by Rainer Tammer.
+
 2010-07-31  Bruno Haible  <bruno@clisp.org>
 
 	linkat: Work around AIX 7.1 bug.
--- a/m4/openat.m4	Sat Jul 31 15:53:21 2010 +0200
+++ b/m4/openat.m4	Sat Jul 31 16:01:58 2010 +0200
@@ -1,4 +1,4 @@
-# serial 29
+# serial 30
 # See if we need to use our replacement for Solaris' openat et al functions.
 
 dnl Copyright (C) 2004-2010 Free Software Foundation, Inc.
@@ -99,19 +99,22 @@
 # If we have the fchownat function, and it has the bug (in glibc-2.4)
 # that it dereferences symlinks even with AT_SYMLINK_NOFOLLOW, then
 # use the replacement function.
+# Also if the fchownat function, like chown, has the trailing slash bug,
+# use the replacement function.
 # Also use the replacement function if fchownat is simply not available.
 AC_DEFUN([gl_FUNC_FCHOWNAT],
 [
   AC_REQUIRE([gl_FUNC_CHOWN])
   AC_CHECK_FUNC([fchownat],
-    [gl_FUNC_FCHOWNAT_DEREF_BUG([
-      REPLACE_FCHOWNAT=1
-      AC_DEFINE([FCHOWNAT_NOFOLLOW_BUG], [1],
-                [Define to 1 if your platform has fchownat, but it cannot
-                 perform lchown tasks.])
-      if test $REPLACE_CHOWN = 1; then
-        REPLACE_FCHOWNAT=1
-      fi])],
+    [gl_FUNC_FCHOWNAT_DEREF_BUG(
+       [REPLACE_FCHOWNAT=1
+        AC_DEFINE([FCHOWNAT_NOFOLLOW_BUG], [1],
+                  [Define to 1 if your platform has fchownat, but it cannot
+                   perform lchown tasks.])
+       ])
+     if test $REPLACE_CHOWN = 1; then
+       REPLACE_FCHOWNAT=1
+     fi],
     [HAVE_FCHOWNAT=0])
   if test $HAVE_FCHOWNAT = 0 || test $REPLACE_FCHOWNAT = 1; then
     AC_LIBOBJ([fchownat])