Handle an invalid AbbrCode in DWARF handling (#2607)

Fixes the testcase in #2343 (comment)

Looks like that's from Rust. Not sure why it would have an invalid
abbreviation code, but perhaps the LLVM there emits dwarf differently
than we've tested on so far. May be worth investigating further, but
for now emit a warning, skip that element, and don't crash.

Also fix valgrind warnings about Span values not being initialized,
which was invalid and bad as well (wasted memory in our maps,
and might have overlapped with real values), and interfered with
figuring this out.

1 files changed, 7 insertions, 0 deletions

diff --git a/third_party/llvm-project/DWARFVisitor.cpp b/third_party/llvm-project/DWARFVisitor.cpp
index 35d5ccf57..05c4194e3 100644
--- a/third_party/llvm-project/DWARFVisitor.cpp
+++ b/third_party/llvm-project/DWARFVisitor.cpp
@@ -55,6 +55,13 @@ template <typename T> void DWARFYAML::VisitorImpl<T>::traverseDebugInfo() {
       onStartDIE(Unit, Entry);
       if (Entry.AbbrCode == 0u)
         continue;
+      // XXX BINARYEN
+      if (Entry.AbbrCode - FirstAbbrevCode >= DebugInfo.AbbrevDecls.size()) {
+        errs() << "warning: invalid abbreviation code " << Entry.AbbrCode <<
+               " (range: " << FirstAbbrevCode << "-" <<
+               (DebugInfo.AbbrevDecls.size() - FirstAbbrevCode) << ")\n";
+        continue;
+      }
       auto &Abbrev = DebugInfo.AbbrevDecls[Entry.AbbrCode - FirstAbbrevCode];
       auto FormVal = Entry.Values.begin();
       auto AbbrForm = Abbrev.Attributes.begin();