diff options
| author | Alan Modra <amodra@gmail.com> | 2015-01-06 16:46:40 +0000 |
|---|---|---|
| committer | Nick Clifton <nickc@redhat.com> | 2015-01-06 16:46:40 +0000 |
| commit | 60ebc25751744f54ae5c00d1c9198ab64fcc5b33 (patch) | |
| tree | c9241d9ee32ae0223125d0467c96271f18570fcb /include/coff | |
| parent | 85880250e591a51624d24db653aaace0c5ce5943 (diff) | |
Fixes a buffer overflow when compiling assembler for the MinGW targets.
PR binutils/17754
* internal.h (internal_auxent): Increase size of x_fname field to
20 to allow for PE format's longer file names.
Diffstat (limited to 'include/coff')
| -rw-r--r-- | include/coff/ChangeLog | 6 | ||||
| -rw-r--r-- | include/coff/internal.h | 6 |
2 files changed, 11 insertions, 1 deletions
diff --git a/include/coff/ChangeLog b/include/coff/ChangeLog index 343298f91b..232e4fca3b 100644 --- a/include/coff/ChangeLog +++ b/include/coff/ChangeLog @@ -1,3 +1,9 @@ +2015-01-06 Alan Modra <amodra@gmail.com> + + PR binutils/17754 + * internal.h (internal_auxent): Increase size of x_fname field to + 20 to allow for PE format's longer file names. + 2015-01-01 Alan Modra <amodra@gmail.com> Update year range in copyright notice of all files. diff --git a/include/coff/internal.h b/include/coff/internal.h index 83295645d4..c4f24bb1e0 100644 --- a/include/coff/internal.h +++ b/include/coff/internal.h @@ -588,7 +588,11 @@ union internal_auxent union { - char x_fname[FILNMLEN]; + /* PR 17754: We use to FILNMLEN for the size of the x_fname + array, but that cause problems as PE targets use a larger + value. We cannot use their definition of EFILNMLEN as this + header can be used without including any PE headers. */ + char x_fname[20]; struct { long x_zeroes; |