diff options
| author | Alan Modra <amodra@gmail.com> | 2015-08-22 13:24:34 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2015-08-22 16:24:52 +0930 |
| commit | 595bce7595e076391e7147b0c6db57a8edf2e406 (patch) | |
| tree | 352470d415a279d64e65d60fa61c3e11f2195a2c /bfd/elf.c | |
| parent | 0d6b12851b08f5f71f7d97d2c832b899d63b5df2 (diff) | |
| download | binutils-gdb-595bce7595e076391e7147b0c6db57a8edf2e406.tar.gz | |
Uninitialized vd_nodename
Belt and braces fix. Either of the changes here is sufficient to
ensure vd_nodename is initialized properly.
* elf.c (_bfd_elf_slurp_version_tables): Always init vd_nodename.
Don't copy fields not set by _bfd_elf_swap_verdef_in.
Diffstat (limited to 'bfd/elf.c')
| -rw-r--r-- | bfd/elf.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/bfd/elf.c b/bfd/elf.c index f20679e303c..f8e12958452 100644 --- a/bfd/elf.c +++ b/bfd/elf.c @@ -7871,7 +7871,7 @@ error_return_verref: goto error_return_bad_verdef; iverdef = &iverdefarr[(iverdefmem.vd_ndx & VERSYM_VERSION) - 1]; - memcpy (iverdef, &iverdefmem, sizeof (Elf_Internal_Verdef)); + memcpy (iverdef, &iverdefmem, offsetof (Elf_Internal_Verdef, vd_bfd)); iverdef->vd_bfd = abfd; @@ -7920,6 +7920,7 @@ error_return_verref: ((bfd_byte *) everdaux + iverdaux->vda_next)); } + iverdef->vd_nodename = NULL; if (iverdef->vd_cnt) iverdef->vd_nodename = iverdef->vd_auxptr->vda_nodename; |