This is the mail archive of the mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[gold] PATCH: PR gold/14993: Section sorting interferes with the incremental update


We set negative section offset in Input_section::set_address_and_file_offset,
when storing section offset as an address. Although it is overridden by
Sized_relobj_file<size, big_endian>::layout_section later, the negative
offset doesn't work on x32 since a section offset (18446744073709551050
== 0xFFFFFFFFFFFFFDCA == -3530) that doesn't fit in a 32-bit Address.
This patch changes negative offset to -1, which is converted to
invalid_address.  Tested it on x86-64, x32 and ia32.  OK to install?


2013-01-03  H.J. Lu  <>

	PR gold/14993
	* (Output_section::Input_section::set_address_and_file_offset):
	Set negative offset to -1.

diff --git a/gold/ b/gold/
index f2321b7..75feb74 100644
--- a/gold/
+++ b/gold/
@@ -2152,8 +2152,12 @@ Output_section::Input_section::set_address_and_file_offset(
     off_t section_file_offset)
   if (this->is_input_section())
-    this->u2_.object->set_section_offset(this->shndx_,
-					 file_offset - section_file_offset);
+    {
+      file_offset -= section_file_offset;
+      if (file_offset < 0)
+	file_offset = static_cast<off_t>(-1);
+      this->u2_.object->set_section_offset(this->shndx_, file_offset);
+    }
     this->u2_.posd->set_address_and_file_offset(address, file_offset);

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]