This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[PATCH v2 3/4] Make sure that sorting does not change section order
- From: Petr Tesarik <ptesarik at suse dot cz>
- To: gdb-patches at sourceware dot org
- Cc: Petr Tesarik <ptesarik at suse dot cz>, Jeff Mahoney <jeffm at suse dot com>
- Date: Mon, 11 Jun 2018 14:08:34 +0200
- Subject: [PATCH v2 3/4] Make sure that sorting does not change section order
- References: <20180611120835.27343-1-ptesarik@suse.cz>
Symbol files may contain multiple sections with the same name.
Section addresses specified add-symbol-file are assigned to the
corresponding BFD sections in addr_info_make_relative using sorted
indexes of both vectors. Since the sort algorithm is not inherently
stable, the comparison function uses sectindex to maintain the
original order. However, add_symbol_file_command uses zero for all
sections, so if the user specifies multiple sections with the same
name, they will be assigned randomly to symbol file sections with
the same name.
gdb/ChangeLog:
2018-06-11 Petr Tesarik <ptesarik@suse.com>
* symfile.c (add_symbol_file_command): Make sure that sections
with the same name are sorted in the same order.
---
gdb/ChangeLog | 5 +++++
gdb/symfile.c | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 1c5a1f6bfb..0f75992d4c 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,10 @@
2018-06-11 Petr Tesarik <ptesarik@suse.com>
+ * symfile.c (add_symbol_file_command): Make sure that sections
+ with the same name are sorted in the same order.
+
+2018-06-11 Petr Tesarik <ptesarik@suse.com>
+
* symfile.c (add_symbol_file_command, _initialize_symfile): Do not
require the second argument. If omitted, load sections at the
addresses specified in the file.
diff --git a/gdb/symfile.c b/gdb/symfile.c
index 3e3ab20412..8b8b194334 100644
--- a/gdb/symfile.c
+++ b/gdb/symfile.c
@@ -2185,7 +2185,7 @@ add_symbol_file_command (const char *args, int from_tty)
/* Here we store the section offsets in the order they were
entered on the command line. */
- section_addrs.emplace_back (addr, sec, 0);
+ section_addrs.emplace_back (addr, sec, section_addrs.size ());
printf_unfiltered ("\t%s_addr = %s\n", sec,
paddress (gdbarch, addr));
--
2.13.6