diff options
author | Nick Clifton <nickc@redhat.com> | 2010-08-13 11:36:10 +0000 |
---|---|---|
committer | Nick Clifton <nickc@redhat.com> | 2010-08-13 11:36:10 +0000 |
commit | 9d8d6261e33d9fc30804baba9de325453cc3092d (patch) | |
tree | d8f6f289b17f8ff98b912e87192d73f6074107d8 /libiberty/argv.c | |
parent | 1d9ec526eeeeb28ba849e65993860f2ef1b750d2 (diff) | |
download | binutils-9d8d6261e33d9fc30804baba9de325453cc3092d.zip binutils-9d8d6261e33d9fc30804baba9de325453cc3092d.tar.gz binutils-9d8d6261e33d9fc30804baba9de325453cc3092d.tar.bz2 |
* argv.c (expandargv): Limit the number of times that response
files are opened in order to prevent infinite recursion.
Diffstat (limited to 'libiberty/argv.c')
-rw-r--r-- | libiberty/argv.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/libiberty/argv.c b/libiberty/argv.c index 3084248..8476c8f 100644 --- a/libiberty/argv.c +++ b/libiberty/argv.c @@ -1,5 +1,5 @@ /* Create and destroy argument vectors (argv's) - Copyright (C) 1992, 2001 Free Software Foundation, Inc. + Copyright (C) 1992, 2001, 2010 Free Software Foundation, Inc. Written by Fred Fish @ Cygnus Support This file is part of the libiberty library. @@ -386,6 +386,9 @@ expandargv (int *argcp, char ***argvp) int i = 0; /* Non-zero if ***argvp has been dynamically allocated. */ int argv_dynamic = 0; + /* Limit the number of response files that we parse in order + to prevent infinite recursion. */ + unsigned int iteration_limit = 2000; /* Loop over the arguments, handling response files. We always skip ARGVP[0], as that is the name of the program being run. */ while (++i < *argcp) @@ -412,6 +415,12 @@ expandargv (int *argcp, char ***argvp) filename = (*argvp)[i]; if (filename[0] != '@') continue; + /* If we have iterated too many times then stop. */ + if (-- iteration_limit == 0) + { + fprintf (stderr, "%s: error: too many @-files encountered\n", (*argvp)[0]); + xexit (1); + } /* Read the contents of the file. */ f = fopen (++filename, "r"); if (!f) |