Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(206)

Unified Diff: src/wasm/wasm-module.cc

Issue 2397303002: Fix build error due to conflicting changes. (Closed)
Patch Set: remove bytes check from validation of wasm module object Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/api.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/wasm/wasm-module.cc
diff --git a/src/wasm/wasm-module.cc b/src/wasm/wasm-module.cc
index fdc34f0e8e34b2abdeda27aef84f154c9b02f00f..58d31d20bd899c67f2a1dd8d8a85ae565b135b38 100644
--- a/src/wasm/wasm-module.cc
+++ b/src/wasm/wasm-module.cc
@@ -1892,13 +1892,6 @@ bool WasmCompiledModule::IsWasmCompiledModule(Object* obj) {
WCM_PROPERTY_TABLE(WCM_CHECK)
#undef WCM_CHECK
- WasmCompiledModule* compiled_module =
- reinterpret_cast<WasmCompiledModule*>(obj);
- if (!compiled_module->has_module_bytes()) return false;
- SeqOneByteString* module_bytes = compiled_module->ptr_to_module_bytes();
- if (module_bytes->length() < 4) return false;
- if (memcmp(module_bytes->GetChars(), "\0asm", 4)) return false;
-
// All checks passed.
return true;
}
« no previous file with comments | « src/api.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698