diff options
| author | Nick Clifton <nickc@redhat.com> | 2021-03-16 12:44:50 +0000 |
|---|---|---|
| committer | Nick Clifton <nickc@redhat.com> | 2021-03-16 12:44:50 +0000 |
| commit | 012d44268695f1c5d4e2e019c610c8b99bd7e553 (patch) | |
| tree | 3edc46955760196226b70e7e73cbf0d9fc904001 /include/coff | |
| parent | 7807d76a1cc193984c1bdfab456df97632ac721f (diff) | |
| download | binutils-012d44268695f1c5d4e2e019c610c8b99bd7e553.zip binutils-012d44268695f1c5d4e2e019c610c8b99bd7e553.tar.gz binutils-012d44268695f1c5d4e2e019c610c8b99bd7e553.tar.bz2 | |
Fix a potential buffer overrun qwhen writing out PE aux entries.
* peXXigen.c (_bfd_XXi_swap_aux_out): Avoid potential buffer
overrun by using sizeof of the destination x_fname field as the
limit for a memcpy.
* coff/internal.h (struct internal_auxent): Fix a couple of typos
in comment describing the x_fname field.
Diffstat (limited to 'include/coff')
| -rw-r--r-- | include/coff/internal.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/coff/internal.h b/include/coff/internal.h index 1b5b45c..90c901d 100644 --- a/include/coff/internal.h +++ b/include/coff/internal.h @@ -580,8 +580,8 @@ union internal_auxent union { /* 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 + array, but that causes problems as PE targets use a larger + value. We cannot use their definition of E_FILNMLEN as this header can be used without including any PE headers. */ char x_fname[20]; struct |