From: Jim Meyering <jim@meyering.net>
Date: Sat, 2 Dec 2000 20:55:02 +0000 (+0000)
Subject: (gobble_file): Do not fall back on lstat if stat
X-Git-Tag: TEXTUTILS-2_0_9~21
X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=e49511faef949ea27e266f11030351aed935dddc;p=thirdparty%2Fcoreutils.git

(gobble_file): Do not fall back on lstat if stat
fails; POSIX.2 does not allow this.  Invoke acl only on
non-symlinks, and only if lstat or stat succeeds.
---

diff --git a/src/ls.c b/src/ls.c
index 146b5aa5d5..6fb10edf55 100644
--- a/src/ls.c
+++ b/src/ls.c
@@ -1887,23 +1887,9 @@ gobble_file (const char *name, enum filetype type, int explicit_arg,
 	  attach (path, dirname, name);
 	}
 
-      if (trace_links)
-	{
-	  val = stat (path, &files[files_index].stat);
-	  if (val < 0)
-	    {
-	      /* Perhaps a symbolically-linked to file doesn't exist; stat
-		 the link instead. */
-	      val = lstat (path, &files[files_index].stat);
-	    }
-	}
-      else
-	{
-	  val = lstat (path, &files[files_index].stat);
-#if USE_ACL
-	  files[files_index].have_acl = (acl (path, GETACLCNT, 0, NULL) > 4);
-#endif
-	}
+      val = (trace_links
+	     ? stat (path, &files[files_index].stat)
+	     : lstat (path, &files[files_index].stat));
 
       if (val < 0)
 	{
@@ -1912,6 +1898,11 @@ gobble_file (const char *name, enum filetype type, int explicit_arg,
 	  return 0;
 	}
 
+#if USE_ACL
+      if (! S_ISLNK (files[files_index].stat.st_mode))
+	files[files_index].have_acl = 4 < acl (path, GETACLCNT, 0, NULL);
+#endif
+
       if (S_ISLNK (files[files_index].stat.st_mode)
 	  && (explicit_arg || format == long_format || check_symlink_color))
 	{