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 /bfd/peXXigen.c | |
| parent | 7807d76a1cc193984c1bdfab456df97632ac721f (diff) | |
| download | gdb-012d44268695f1c5d4e2e019c610c8b99bd7e553.zip gdb-012d44268695f1c5d4e2e019c610c8b99bd7e553.tar.gz gdb-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 'bfd/peXXigen.c')
| -rw-r--r-- | bfd/peXXigen.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bfd/peXXigen.c b/bfd/peXXigen.c index 83bbac5..b370672 100644 --- a/bfd/peXXigen.c +++ b/bfd/peXXigen.c @@ -380,7 +380,7 @@ _bfd_XXi_swap_aux_out (bfd * abfd, H_PUT_32 (abfd, in->x_file.x_n.x_offset, ext->x_file.x_n.x_offset); } else - memcpy (ext->x_file.x_fname, in->x_file.x_fname, FILNMLEN); + memcpy (ext->x_file.x_fname, in->x_file.x_fname, sizeof (ext->x_file.x_fname)); return AUXESZ; |