From: Cary Coutant <ccoutant@google.com>
Date: Tue, 13 Mar 2012 00:25:58 +0000 (+0000)
Subject: 	* dwarf_reader.cc (Sized_dwarf_line_info::process_one_opcode): Fix
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e8dd54e14a498292154a6511c03d8953379b7688;p=binutils-gdb.git

	* dwarf_reader.cc (Sized_dwarf_line_info::process_one_opcode): Fix
	handling of DW_LNE_define_file.
---

diff --git a/gold/ChangeLog b/gold/ChangeLog
index f0ec8938f2a..0a72132fe02 100644
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,8 @@
+2012-03-12  Cary Coutant  <ccoutant@google.com>
+
+	* dwarf_reader.cc (Sized_dwarf_line_info::process_one_opcode): Fix
+	handling of DW_LNE_define_file.
+
 2012-03-12  Cary Coutant  <ccoutant@google.com>
 
 	* reduced_debug_output.cc
diff --git a/gold/dwarf_reader.cc b/gold/dwarf_reader.cc
index 73f84b0d420..189e6a6740f 100644
--- a/gold/dwarf_reader.cc
+++ b/gold/dwarf_reader.cc
@@ -413,17 +413,16 @@ Sized_dwarf_line_info<size, big_endian>::process_one_opcode(
               start += templen;
 
               uint64_t dirindex = read_unsigned_LEB_128(start, &templen);
-              oplen += templen;
 
               if (dirindex >= this->directories_.back().size())
                 dirindex = 0;
 	      int dirindexi = static_cast<int>(dirindex);
 
-              read_unsigned_LEB_128(start, &templen);   // mod_time
-              oplen += templen;
-
-              read_unsigned_LEB_128(start, &templen);   // filelength
-              oplen += templen;
+              // This opcode takes two additional ULEB128 parameters
+              // (mod_time and filelength), but we don't use those
+              // values.  Because OPLEN already tells us how far to
+              // skip to the next opcode, we don't need to read
+              // them at all.
 
               this->files_.back().push_back(std::make_pair(dirindexi,
 							   filename));