diff options
| author | Alan Modra <amodra@gmail.com> | 2021-11-05 21:26:16 +1030 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2021-11-05 21:32:02 +1030 |
| commit | ca8775833bb6af8eb4333f41e4f8672df0ead874 (patch) | |
| tree | baec327f0c07ef4f4b056380ec752936df9a74ed /binutils | |
| parent | 109c1107c4394f0e85fa905ecb369f40d36fbef4 (diff) | |
| download | fsf-binutils-gdb-ca8775833bb6af8eb4333f41e4f8672df0ead874.zip fsf-binutils-gdb-ca8775833bb6af8eb4333f41e4f8672df0ead874.tar.gz fsf-binutils-gdb-ca8775833bb6af8eb4333f41e4f8672df0ead874.tar.bz2 | |
PR28541, unstable cie offset in the output of readelf
Calculating "0 - pointer" can indeed result in seeming randomness as
the pointer address varies.
PR 28541
* dwarf.c (display_debug_frames): Don't print cie offset when
invalid, print "invalid" instead. Remove now redundant warning.
Diffstat (limited to 'binutils')
| -rw-r--r-- | binutils/dwarf.c | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/binutils/dwarf.c b/binutils/dwarf.c index 9fed113..d80235f 100644 --- a/binutils/dwarf.c +++ b/binutils/dwarf.c @@ -8777,9 +8777,6 @@ display_debug_frames (struct dwarf_section *section, if (!cie) { - warn ("Invalid CIE pointer 0x%s in FDE at %#08lx\n", - dwarf_vmatoa_1 (NULL, cie_id, offset_size), - (unsigned long) (saved_start - section_start)); fc->ncols = 0; fc->col_type = (short int *) xmalloc (sizeof (short int)); fc->col_offset = (int *) xmalloc (sizeof (int)); @@ -8862,12 +8859,20 @@ display_debug_frames (struct dwarf_section *section, start += augmentation_data_len; } - printf ("\n%08lx %s %s FDE cie=%08lx pc=", + printf ("\n%08lx %s %s FDE ", (unsigned long)(saved_start - section_start), dwarf_vmatoa_1 (NULL, length, fc->ptr_size), - dwarf_vmatoa_1 (NULL, cie_id, offset_size), - (unsigned long)(cie->chunk_start - section_start)); + dwarf_vmatoa_1 (NULL, cie_id, offset_size)); + if (cie->chunk_start) + printf ("cie=%08lx", + (unsigned long) (cie->chunk_start - section_start)); + else + /* Ideally translate "invalid " to 8 chars, trailing space + is optional. */ + printf (_("cie=invalid ")); + + printf (" pc="); if (fc->segment_size) printf ("%04lx:", segment_selector); |