From: Cary Coutant <ccoutant@google.com>
Date: Wed, 25 Jun 2008 17:10:08 +0000 (+0000)
Subject: 2008-06-25  Cary Coutant  <ccoutant@google.com>
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=3f2e6a2da5d9348efaa98eb06c430bec5fbe596b;p=binutils-gdb.git

2008-06-25  Cary Coutant  <ccoutant@google.com>

	* fileread.cc (File_read::make_view): Assert on zero-length view.
	* object.cc (Sized_relobj::do_read_symbols): Don't try to read
	symbol table when there are no symbols to read.
---

diff --git a/gold/ChangeLog b/gold/ChangeLog
index 22f549b36d4..b8d811866ff 100644
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,9 @@
+2008-06-25  Cary Coutant  <ccoutant@google.com>
+
+	* fileread.cc (File_read::make_view): Assert on zero-length view.
+	* object.cc (Sized_relobj::do_read_symbols): Don't try to read
+	symbol table when there are no symbols to read.
+
 2008-06-23  Craig Silverstein  <csilvers@google.com>
 
 	* version.cc (version_string): Bump to 1.7
diff --git a/gold/fileread.cc b/gold/fileread.cc
index 6cb056fd1d2..cfe0ee6debb 100644
--- a/gold/fileread.cc
+++ b/gold/fileread.cc
@@ -327,6 +327,8 @@ File_read::View*
 File_read::make_view(off_t start, section_size_type size,
 		     unsigned int byteshift, bool cache)
 {
+  gold_assert(size > 0);
+
   off_t poff = File_read::page_offset(start);
 
   section_size_type psize = File_read::pages(size + (start - poff));
diff --git a/gold/object.cc b/gold/object.cc
index b49bee17502..8659cb24ba5 100644
--- a/gold/object.cc
+++ b/gold/object.cc
@@ -452,6 +452,12 @@ Sized_relobj<size, big_endian>::do_read_symbols(Read_symbols_data* sd)
   off_t readoff = this->has_eh_frame_ ? dataoff : extoff;
   section_size_type readsize = this->has_eh_frame_ ? datasize : extsize;
 
+  if (readsize == 0)
+    {
+      // No external symbols.  Also weird but also legal.
+      return;
+    }
+
   File_view* fvsymtab = this->get_lasting_view(readoff, readsize, true, false);
 
   // Read the section header for the symbol names.