[wasm] test deserialization when header is invalid

test/cctest/wasm/test-run-wasm-module.cc

Index: test/cctest/wasm/test-run-wasm-module.cc
diff --git a/test/cctest/wasm/test-run-wasm-module.cc b/test/cctest/wasm/test-run-wasm-module.cc
index 8a45955b8dafd9b5e4b500ce905f71ffe3396b27..104eed7684ce3263fbea216ac1adb13ed1c84784 100644
--- a/test/cctest/wasm/test-run-wasm-module.cc
+++ b/test/cctest/wasm/test-run-wasm-module.cc
@@ -227,10 +227,16 @@ TEST(Run_WasmModule_Serialization) {
   create_params.array_buffer_allocator =
       CcTest::InitIsolateOnce()->array_buffer_allocator();
 
-  for (int i = 0; i < 2; ++i) {
+  for (int i = 0; i < 3; ++i) {
     v8::Isolate* v8_isolate = v8::Isolate::New(create_params);
     if (i == 1) {
-      // Mess with the serialized data to force recompilation.
+      // Mess the header.

[bradnelson, 2016/10/12 19:10:25]

Mess with?

[Mircea Trofin, 2016/10/12 20:32:06]

Done.

+      DCHECK_NE(0, data.first.get()[0]);
+      const_cast<uint8_t*>(data.first.get())[0] = 0;

[bradnelson, 2016/10/12 19:10:25]

Kinda bad. Maybe create an accessor that's a friend on a test fixture?

[Mircea Trofin, 2016/10/12 20:32:06]

Done.

+    }
+
+    if (i == 2) {
+      // Reset the compiled module data.
       data.first.reset();
       data.second = 0;
     }