diff options
| author | Jan Beulich <jbeulich@suse.com> | 2021-03-10 08:12:37 +0100 |
|---|---|---|
| committer | Jan Beulich <jbeulich@suse.com> | 2021-03-10 08:12:37 +0100 |
| commit | 67b0f684803f5bec9b801c43fe1559bc88c15bb6 (patch) | |
| tree | 6430680e12759db6270ac8b4ffde38bab1756605 /bfd/cofflink.c | |
| parent | 3044062c34f2372f4ecf81b69d9e63685cd5e995 (diff) | |
| download | fsf-binutils-gdb-67b0f684803f5bec9b801c43fe1559bc88c15bb6.zip fsf-binutils-gdb-67b0f684803f5bec9b801c43fe1559bc88c15bb6.tar.gz fsf-binutils-gdb-67b0f684803f5bec9b801c43fe1559bc88c15bb6.tar.bz2 | |
bfd: strip symbols not representable in COFF/PE symbol table
The offset-within-section field in the symbol table entry is only 32
bits wide, so rather than emitting bogus entries omit them, and issue
a diagnostic identifying the issue.
This requires adjusting the PR/22267 test to no longer produce symbols
with out of range values on 64-bit BFD. This also depends on
adjustments to testsuite/ld-scripts/map-address.* made by an earlier
patch. The purpose of the test can very well be achieved nevertheless.
Diffstat (limited to 'bfd/cofflink.c')
| -rw-r--r-- | bfd/cofflink.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/bfd/cofflink.c b/bfd/cofflink.c index 4b0aab2..874354a 100644 --- a/bfd/cofflink.c +++ b/bfd/cofflink.c @@ -2602,6 +2602,17 @@ _bfd_coff_write_global_sym (struct bfd_hash_entry *bh, void *data) + h->root.u.def.section->output_offset); if (! obj_pe (flaginfo->output_bfd)) isym.n_value += sec->vma; +#ifdef BFD64 + if (isym.n_value > (bfd_vma) 0xffffffff) + { + if (! h->root.linker_def) + _bfd_error_handler + (_("%pB: stripping non-representable symbol '%s' (value " + "%" BFD_VMA_FMT "x)"), + output_bfd, h->root.root.string, isym.n_value); + return TRUE; + } +#endif } break; |