argv.c (expandargv): Check for directories passed as @-files.
authorDJ Delorie <dj@redhat.com>
Tue, 6 Dec 2016 06:40:44 +0000 (01:40 -0500)
committerDJ Delorie <dj@redhat.com>
Tue, 6 Dec 2016 06:40:44 +0000 (01:40 -0500)
gcc pr 78584

libiberty/ChangeLog
libiberty/argv.c

index 7ae91adb947520206455e0f51b4f04a21400bdef..f800e0c0a25ca4a4b8d9190dbaf2ab3d3ed927d8 100644 (file)
@@ -1,3 +1,7 @@
+2016-12-06  DJ Delorie  <dj@redhat.com>
+
+       * argv.c (expandargv): Check for directories passed as @-files.
+
 2016-11-03  David Tolnay <dtolnay@gmail.com>
            Mark Wielaard  <mark@klomp.org>
 
index 994dd3531c00952f79dbdabf4341cbea85ac22c8..fd43e78b170fbf71eebfc269fcf1947b8175227a 100644 (file)
@@ -35,6 +35,13 @@ Boston, MA 02110-1301, USA.  */
 #include <string.h>
 #include <stdlib.h>
 #include <stdio.h>
+#include <sys/types.h>
+#ifdef HAVE_UNISTD_H
+#include <unistd.h>
+#endif
+#if HAVE_SYS_STAT_H
+#include <sys/stat.h>
+#endif
 
 #ifndef NULL
 #define NULL 0
@@ -387,6 +394,9 @@ expandargv (int *argcp, char ***argvp)
       char **file_argv;
       /* The number of options read from the response file, if any.  */
       size_t file_argc;
+#ifdef S_ISDIR
+      struct stat sb;
+#endif
       /* We are only interested in options of the form "@file".  */
       filename = (*argvp)[i];
       if (filename[0] != '@')
@@ -397,6 +407,15 @@ expandargv (int *argcp, char ***argvp)
          fprintf (stderr, "%s: error: too many @-files encountered\n", (*argvp)[0]);
          xexit (1);
        }
+#ifdef S_ISDIR
+      if (stat (filename+1, &sb) < 0)
+       continue;
+      if (S_ISDIR(sb.st_mode))
+       {
+         fprintf (stderr, "%s: error: @-file refers to a directory\n", (*argvp)[0]);
+         xexit (1);
+       }
+#endif
       /* Read the contents of the file.  */
       f = fopen (++filename, "r");
       if (!f)