aboutsummaryrefslogtreecommitdiff
path: root/libiberty/argv.c
diff options
context:
space:
mode:
authorDJ Delorie <dj@redhat.com>2006-01-21 04:28:49 +0000
committerDJ Delorie <dj@redhat.com>2006-01-21 04:28:49 +0000
commit67d78caad5228c04e198e8e5d96dc27fdc28adb9 (patch)
tree398b5d170ffffe464c4c9ea708ce0101e3946b70 /libiberty/argv.c
parentacab6ab29b26848dc49786d1b488961038bf99b4 (diff)
downloadgdb-67d78caad5228c04e198e8e5d96dc27fdc28adb9.zip
gdb-67d78caad5228c04e198e8e5d96dc27fdc28adb9.tar.gz
gdb-67d78caad5228c04e198e8e5d96dc27fdc28adb9.tar.bz2
merge from gcc
Diffstat (limited to 'libiberty/argv.c')
-rw-r--r--libiberty/argv.c17
1 files changed, 13 insertions, 4 deletions
diff --git a/libiberty/argv.c b/libiberty/argv.c
index 79241b6..11ca549 100644
--- a/libiberty/argv.c
+++ b/libiberty/argv.c
@@ -328,8 +328,12 @@ expandargv (argcp, argvp)
const char *filename;
/* The response file. */
FILE *f;
- /* The number of characters in the response file. */
+ /* An upper bound on the number of characters in the response
+ file. */
long pos;
+ /* The number of characters in the response file, when actually
+ read. */
+ size_t len;
/* A dynamically allocated buffer used to hold options read from a
response file. */
char *buffer;
@@ -337,7 +341,7 @@ expandargv (argcp, argvp)
response file. */
char **file_argv;
/* The number of options read from the response file, if any. */
- size_t file_argc;
+ size_t file_argc;
/* We are only interested in options of the form "@file". */
filename = (*argvp)[i];
if (filename[0] != '@')
@@ -354,10 +358,15 @@ expandargv (argcp, argvp)
if (fseek (f, 0L, SEEK_SET) == -1)
goto error;
buffer = (char *) xmalloc (pos * sizeof (char) + 1);
- if (fread (buffer, sizeof (char), pos, f) != (size_t) pos)
+ len = fread (buffer, sizeof (char), pos, f);
+ if (len != (size_t) pos
+ /* On Windows, fread may return a value smaller than POS,
+ due to CR/LF->CR translation when reading text files.
+ That does not in-and-of itself indicate failure. */
+ && ferror (f))
goto error;
/* Add a NUL terminator. */
- buffer[pos] = '\0';
+ buffer[len] = '\0';
/* Parse the string. */
file_argv = buildargv (buffer);
/* If *ARGVP is not already dynamically allocated, copy it. */