Fix linking section symbol name bugs (#1508)

Found by oss-fuzz.

1 files changed, 16 insertions, 0 deletions

diff --git a/src/binary-reader-ir.cc b/src/binary-reader-ir.cc
index 0dc9b2b1..3b0fc453 100644
--- a/src/binary-reader-ir.cc
+++ b/src/binary-reader-ir.cc
@@ -1277,6 +1277,10 @@ Result BinaryReaderIR::OnDataSymbol(Index index, uint32_t flags,
     // the whole segment.
     return Result::Ok;
   }
+  if (segment >= module_->data_segments.size()) {
+    PrintError("invalid data segment index: %" PRIindex, segment);
+    return Result::Error;
+  }
   DataSegment* seg = module_->data_segments[segment];
   std::string dollar_name =
       GetUniqueName(&module_->data_segment_bindings, MakeDollarName(name));
@@ -1290,6 +1294,10 @@ Result BinaryReaderIR::OnFunctionSymbol(Index index, uint32_t flags,
   if (name.empty()) {
     return Result::Ok;
   }
+  if (func_index >= module_->funcs.size()) {
+    PrintError("invalid function index: %" PRIindex, func_index);
+    return Result::Error;
+  }
   Func* func = module_->funcs[func_index];
   if (!func->name.empty()) {
     // The name section has already named this function.
@@ -1307,6 +1315,10 @@ Result BinaryReaderIR::OnGlobalSymbol(Index index, uint32_t flags,
   if (name.empty()) {
     return Result::Ok;
   }
+  if (global_index >= module_->globals.size()) {
+    PrintError("invalid global index: %" PRIindex, global_index);
+    return Result::Error;
+  }
   Global* glob = module_->globals[global_index];
   std::string dollar_name =
       GetUniqueName(&module_->global_bindings, MakeDollarName(name));
@@ -1325,6 +1337,10 @@ Result BinaryReaderIR::OnEventSymbol(Index index, uint32_t flags,
   if (name.empty()) {
     return Result::Ok;
   }
+  if (event_index >= module_->events.size()) {
+    PrintError("invalid event index: %" PRIindex, event_index);
+    return Result::Error;
+  }
   Event* event = module_->events[event_index];
   std::string dollar_name =
       GetUniqueName(&module_->event_bindings, MakeDollarName(name));