configure.in: Add check for lstat.
authorNeil Booth <neil@daikokuya.demon.co.uk>
Fri, 6 Apr 2001 07:22:01 +0000 (07:22 +0000)
committerNeil Booth <neil@gcc.gnu.org>
Fri, 6 Apr 2001 07:22:01 +0000 (07:22 +0000)
* configure.in: Add check for lstat.
* configure, config.in: Regenerate.
* cppinit.c (append_include_chain): Make empty path ".".
        * cpplib.c (do_line): Don't simplify #line paths.
        * cppfiles.c (remove_component_p): New function.
(find_or_create_entry): Acknowledge stat () errors during
path simplification.
(handle_missing_header): Don't simplify paths.
        (_cpp_simplify_pathname): Don't simplify VMS paths.  Return
        the empty path untouched.  Don't leave a trailing '/'.

From-SVN: r41148

gcc/ChangeLog
gcc/config.in
gcc/configure
gcc/configure.in
gcc/cppfiles.c
gcc/cppinit.c
gcc/cpplib.c

index c6165632a63dd7e9617846c792b13794f5793c4d..bb179c6733ae868b476cffa40e457c4272ce8e3d 100644 (file)
@@ -1,3 +1,16 @@
+2001-04-06  Neil Booth  <neil@daikokuya.demon.co.uk>
+
+       * configure.in: Add check for lstat.
+       * configure, config.in: Regenerate.
+       * cppinit.c (append_include_chain): Make empty path ".".
+        * cpplib.c (do_line): Don't simplify #line paths.
+        * cppfiles.c (remove_component_p): New function.
+       (find_or_create_entry): Acknowledge stat () errors during
+       path simplification.
+       (handle_missing_header): Don't simplify paths.
+        (_cpp_simplify_pathname): Don't simplify VMS paths.  Return
+        the empty path untouched.  Don't leave a trailing '/'.
+
 2001-04-06  Benjamin Kosnik  <bkoz@redhat.com>
        
        * cppdefault.c (GPLUSPLUS_BACKWARD_INCLUDE_DIR): Add.
index 7af8d1e818e7be7bad46b8abd55b0f9ed7b94deb..bc19ffe264282dabaeefd25a7f63bc5ef6fb1a3c 100644 (file)
 /* Define if you have the kill function.  */
 #undef HAVE_KILL
 
+/* Define if you have the lstat function.  */
+#undef HAVE_LSTAT
+
 /* Define if you have the munmap function.  */
 #undef HAVE_MUNMAP
 
index 8b0d1b8ef7f994e36222b109ff6131773d5a852d..20383fec2981e079f95b54d83496837b3dfe56c9 100755 (executable)
@@ -3102,7 +3102,7 @@ fi
 for ac_func in strtoul bsearch popen \
        strchr strrchr kill getrlimit setrlimit atoll atoq \
        sysconf isascii gettimeofday strsignal putc_unlocked fputc_unlocked \
-       fputs_unlocked getrusage iconv nl_langinfo
+       fputs_unlocked getrusage iconv nl_langinfo lstat
 do
 echo $ac_n "checking for $ac_func""... $ac_c" 1>&6
 echo "configure:3109: checking for $ac_func" >&5
index ca356f30cc3eb5a7929c796d76af745b453528e7..1e5f408896445d925ee9d2dd38c6af1a39a00bbf 100644 (file)
@@ -547,7 +547,7 @@ dnl gcc_AC_C_ENUM_BF_UNSIGNED
 AC_CHECK_FUNCS(strtoul bsearch popen \
        strchr strrchr kill getrlimit setrlimit atoll atoq \
        sysconf isascii gettimeofday strsignal putc_unlocked fputc_unlocked \
-       fputs_unlocked getrusage iconv nl_langinfo)
+       fputs_unlocked getrusage iconv nl_langinfo lstat)
 
 AC_CHECK_TYPE(ssize_t, int)
 
index bff04ae5326ca887d55295724ded6d6216797260..e242e920f22c50b2f5d5ec08c0ffd8d178cc46b3 100644 (file)
@@ -101,6 +101,7 @@ static int report_missing_guard             PARAMS ((splay_tree_node, void *));
 static splay_tree_node find_or_create_entry PARAMS ((cpp_reader *,
                                                     const char *));
 static void handle_missing_header PARAMS ((cpp_reader *, const char *, int));
+static int remove_component_p  PARAMS ((const char *));
 
 /* Set up the splay tree we use to store information about all the
    file names seen in this compilation.  We also have entries for each
@@ -155,7 +156,8 @@ _cpp_never_reread (file)
 }
 
 /* Lookup a filename, which is simplified after making a copy, and
-   create an entry if none exists.  */
+   create an entry if none exists.  errno is nonzero iff a (reported)
+   stat() error occurred during simplification.  */
 static splay_tree_node
 find_or_create_entry (pfile, fname)
      cpp_reader *pfile;
@@ -208,6 +210,9 @@ open_file (pfile, filename)
   splay_tree_node nd = find_or_create_entry (pfile, filename);
   struct include_file *file = (struct include_file *) nd->value;
 
+  if (errno)
+    file->fd = -2;
+
   /* Don't retry opening if we failed previously.  */
   if (file->fd == -2)
     return 0;
@@ -643,7 +648,6 @@ handle_missing_header (pfile, fname, angle_brackets)
              p[len++] = '/';
            }
          memcpy (p + len, fname, fname_len + 1);
-         _cpp_simplify_pathname (p);
          deps_add_dep (pfile->deps, p);
        }
     }
@@ -1006,6 +1010,26 @@ remap_filename (pfile, name, loc)
   return name;
 }
 
+/* Returns true if it is safe to remove the final component of path,
+   when it is followed by a ".." component.  We use lstat to avoid
+   symlinks if we have it.  If not, we can still catch errors with
+   stat ().  */
+static int
+remove_component_p (path)
+     const char *path;
+{
+  struct stat s;
+  int result;
+
+#ifdef HAVE_LSTAT
+  result = lstat (path, &s);
+#else
+  result = stat (path, &s);
+#endif
+
+  return result == 0 && S_ISDIR (s.st_mode);
+}
+
 /* Simplify a path name in place, deleting redundant components.  This
    reduces OS overhead and guarantees that equivalent paths compare
    the same (modulo symlinks).
@@ -1017,124 +1041,122 @@ remap_filename (pfile, name, loc)
    /../quux            /quux
    //quux              //quux  (POSIX allows leading // as a namespace escape)
 
-   Guarantees no trailing slashes. All transforms reduce the length
-   of the string.  Returns PATH;
- */
+   Guarantees no trailing slashes.  All transforms reduce the length
+   of the string.  Returns PATH.  errno is 0 if no error occurred;
+   nonzero if an error occurred when using stat () or lstat ().  */
+
 char *
 _cpp_simplify_pathname (path)
     char *path;
 {
-    char *from, *to;
-    char *base;
-    int absolute = 0;
+#ifndef VMS
+  char *from, *to;
+  char *base, *orig_base;
+  int absolute = 0;
+
+  errno = 0;
+  /* Don't overflow the empty path by putting a '.' in it below.  */
+  if (*path == '\0')
+    return path;
 
 #if defined (HAVE_DOS_BASED_FILE_SYSTEM)
-    /* Convert all backslashes to slashes. */
-    for (from = path; *from; from++)
-       if (*from == '\\') *from = '/';
+  /* Convert all backslashes to slashes. */
+  for (from = path; *from; from++)
+    if (*from == '\\') *from = '/';
     
-    /* Skip over leading drive letter if present. */
-    if (ISALPHA (path[0]) && path[1] == ':')
-       from = to = &path[2];
-    else
-       from = to = path;
-#else
+  /* Skip over leading drive letter if present. */
+  if (ISALPHA (path[0]) && path[1] == ':')
+    from = to = &path[2];
+  else
     from = to = path;
+#else
+  from = to = path;
 #endif
     
-    /* Remove redundant initial /s.  */
-    if (*from == '/')
+  /* Remove redundant leading /s.  */
+  if (*from == '/')
     {
-       absolute = 1;
-       to++;
-       from++;
-       if (*from == '/')
+      absolute = 1;
+      to++;
+      from++;
+      if (*from == '/')
        {
-           if (*++from == '/')
-               /* 3 or more initial /s are equivalent to 1 /.  */
-               while (*++from == '/');
-           else
-               /* On some hosts // differs from /; Posix allows this.  */
-               to++;
+         if (*++from == '/')
+           /* 3 or more initial /s are equivalent to 1 /.  */
+           while (*++from == '/');
+         else
+           /* On some hosts // differs from /; Posix allows this.  */
+           to++;
        }
     }
-    base = to;
-    
-    for (;;)
+
+  base = orig_base = to;
+  for (;;)
     {
-       while (*from == '/')
-           from++;
-
-       if (from[0] == '.' && from[1] == '/')
-           from += 2;
-       else if (from[0] == '.' && from[1] == '\0')
-           goto done;
-       else if (from[0] == '.' && from[1] == '.' && from[2] == '/')
+      int move_base = 0;
+
+      while (*from == '/')
+       from++;
+
+      if (*from == '\0')
+       break;
+
+      if (*from == '.')
        {
-           if (base == to)
-           {
-               if (absolute)
-                   from += 3;
-               else
-               {
-                   *to++ = *from++;
-                   *to++ = *from++;
-                   *to++ = *from++;
-                   base = to;
-               }
-           }
-           else
+         if (from[1] == '\0')
+           break;
+         if (from[1] == '/')
            {
-               to -= 2;
-               while (to > base && *to != '/') to--;
-               if (*to == '/')
-                   to++;
-               from += 3;
+             from += 2;
+             continue;
            }
-       }
-       else if (from[0] == '.' && from[1] == '.' && from[2] == '\0')
-       {
-           if (base == to)
+         else if (from[1] == '.' && (from[2] == '/' || from[2] == '\0'))
            {
-               if (!absolute)
+             /* Don't simplify if there was no previous component.  */
+             if (absolute && orig_base == to)
                {
-                   *to++ = *from++;
-                   *to++ = *from++;
+                 from += 2;
+                 continue;
                }
-           }
-           else
-           {
-               to -= 2;
-               while (to > base && *to != '/') to--;
-               if (*to == '/')
-                   to++;
-           }
-           goto done;
-       }
-       else
-           /* Copy this component and trailing /, if any.  */
-           while ((*to++ = *from++) != '/')
-           {
-               if (!to[-1])
+             /* Don't simplify if the previous component was "../",
+                or if an error has already occurred with (l)stat.  */
+             if (base != to && errno == 0)
                {
-                   to--;
-                   goto done;
+                 /* We don't back up if it's a symlink.  */
+                 *to = '\0';
+                 if (remove_component_p (path))
+                   {
+                     while (to > base && *to != '/')
+                       to--;
+                     from += 2;
+                     continue;
+                   }
                }
+             move_base = 1;
            }
-       
+       }
+
+      /* Add the component separator.  */
+      if (to > orig_base)
+       *to++ = '/';
+
+      /* Copy this component until the trailing null or '/'.  */
+      while (*from != '\0' && *from != '/')
+       *to++ = *from++;
+
+      if (move_base)
+       base = to;
     }
     
- done:
-    /* Trim trailing slash */
-    if (to[0] == '/' && (!absolute || to > path+1))
-       to--;
-
-    /* Change the empty string to "." so that stat() on the result
-       will always work. */
-    if (to == path)
-      *to++ = '.';
-    
-    *to = '\0';
-
-    return path;
+  /* Change the empty string to "." so that it is not treated as stdin.
+     Null terminate.  */
+  if (to == path)
+    *to++ = '.';
+  *to = '\0';
+
+  return path;
+#else /* VMS  */
+  errno = 0;
+  return path;
+#endif /* !VMS  */
 }
index 13a0e645b92531940aedd93571c10e3a5968ad1c..980cdb1df42e6b875c8d1c7b2aa665b04fd813e2 100644 (file)
@@ -211,10 +211,12 @@ append_include_chain (pfile, dir, path, cxx_aware)
   struct stat st;
   unsigned int len;
 
+  if (*dir == '\0')
+    dir = xstrdup (".");
   _cpp_simplify_pathname (dir);
   if (stat (dir, &st))
     {
-      /* Dirs that don't exist are silently ignored. */
+      /* Dirs that don't exist are silently ignored.  */
       if (errno != ENOENT)
        cpp_notice_from_errno (pfile, dir);
       else if (CPP_OPTION (pfile, verbose))
index 5a523e034a1038d3d9adfbe8712f9ac4208419bc..56ffbb17bc739244d07d582f5a211450d617f2cd 100644 (file)
@@ -729,12 +729,7 @@ do_line (pfile)
   cpp_get_token (pfile, &token);
   if (token.type == CPP_STRING)
     {
-      char *fname;
-      unsigned int len = token.val.str.len + 1;
-
-      fname = (char *) _cpp_pool_alloc (&pfile->ident_pool, len);
-      memcpy (fname, token.val.str.text, len);
-      _cpp_simplify_pathname (fname);
+      const char *fname = (const char *) token.val.str.text;
 
       /* Only accept flags for the # 55 form.  */
       if (! pfile->state.line_extension)