Revert of Merged: [wasm] Fix check failure on invalid name section

src/wasm/module-decoder.cc

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/wasm/module-decoder.h"
 
 #include "src/base/functional.h"
 #include "src/base/platform/platform.h"
 #include "src/flags.h"
 #include "src/macro-assembler.h"
@@ skipping 597 matching lines @@
     }
 
     // ===== Name section ====================================================
     if (section_iter.section_code() == kNameSectionCode) {
       uint32_t functions_count = consume_u32v("functions count");
 
       for (uint32_t i = 0; ok() && i < functions_count; ++i) {
         uint32_t function_name_length = 0;
         uint32_t name_offset = consume_string(&function_name_length, false);
         uint32_t func_index = i;
-        if (inner.ok() && func_index < module->functions.size()) {
+        if (func_index < module->functions.size()) {
           module->functions[func_index].name_offset = name_offset;
           module->functions[func_index].name_length = function_name_length;
         }
 
         uint32_t local_names_count = consume_u32v("local names count");
         for (uint32_t j = 0; ok() && j < local_names_count; j++) {
           skip_string();
         }
       }
       section_iter.advance();
@@ skipping 616 matching lines @@
     table.push_back(std::move(func_asm_offsets));
   }
   if (decoder.more()) decoder.error("unexpected additional bytes");
 
   return decoder.toResult(std::move(table));
 }
 
 }  // namespace wasm
 }  // namespace internal
 }  // namespace v8