aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIan Lance Taylor <ian@airs.com>2008-04-23 17:40:19 +0000
committerIan Lance Taylor <ian@airs.com>2008-04-23 17:40:19 +0000
commit554387026911a74af6e23cd8791d16ce5a1a0070 (patch)
tree88444e4783b5e2ebed088c9d9be09c96430efabd
parentc165fb930adf54d39f41bf4f1da0dac30efcaa8e (diff)
downloadbinutils-554387026911a74af6e23cd8791d16ce5a1a0070.zip
binutils-554387026911a74af6e23cd8791d16ce5a1a0070.tar.gz
binutils-554387026911a74af6e23cd8791d16ce5a1a0070.tar.bz2
* object.cc (Sized_relobj::include_section_group): Check for
invalid section group.
-rw-r--r--gold/ChangeLog3
-rw-r--r--gold/object.cc7
2 files changed, 10 insertions, 0 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog
index 3e3f4fd..b73da9b 100644
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,5 +1,8 @@
2008-04-23 Ian Lance Taylor <iant@google.com>
+ * object.cc (Sized_relobj::include_section_group): Check for
+ invalid section group.
+
* object.cc (make_elf_object): Correct test for 64-bit ELF file
header size.
diff --git a/gold/object.cc b/gold/object.cc
index 9cfd886..1672225 100644
--- a/gold/object.cc
+++ b/gold/object.cc
@@ -609,6 +609,13 @@ Sized_relobj<size, big_endian>::include_section_group(
secnum, index);
continue;
}
+
+ // Check for an earlier section number, since we're going to get
+ // it wrong--we may have already decided to include the section.
+ if (secnum < index)
+ this->error(_("invalid section group %u refers to earlier section %u"),
+ index, secnum);
+
(*omit)[secnum] = true;
}