# HG changeset patch
# User Jim Meyering <meyering@redhat.com>
# Date 1210425311 -7200
# Node ID 48bfcbd671a9b39b4efd53a62883d6b20e9de5ac
# Parent  cfa44ca8e7ba9cefff2a2378769eb87000ec04b1
Make chdir-safer.c slightly safer.

* lib/chdir-safer.c (chdir_no_follow): Test HAVE_WORKING_O_NOFOLLOW,
not O_NOFOLLOW, in case the latter is nonzero and open ignores it.

diff --git a/ChangeLog b/ChangeLog
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2008-05-10  Jim Meyering  <meyering@redhat.com>
 
+	Make chdir-safer.c slightly safer.
+	* lib/chdir-safer.c (chdir_no_follow): Test HAVE_WORKING_O_NOFOLLOW,
+	not O_NOFOLLOW, in case the latter is nonzero and open ignores it.
+
 	Avoid compile failure on systems without ELOOP (like mingw).
 	* lib/chdir-safer.c (ELOOP): Define if not already defined.
 	Reported by Bruno Haible.
diff --git a/lib/chdir-safer.c b/lib/chdir-safer.c
--- a/lib/chdir-safer.c
+++ b/lib/chdir-safer.c
@@ -51,7 +51,7 @@
      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.  */
-  if (! O_NOFOLLOW)
+  if (! HAVE_WORKING_O_NOFOLLOW)
     {
       struct stat sb1;
       result = lstat (dir, &sb1);