We don't properly parse init_expr when the opcode is an unexpected opcode
authorsbarati@apple.com <sbarati@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Jun 2017 02:59:54 +0000 (02:59 +0000)
committersbarati@apple.com <sbarati@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Jun 2017 02:59:54 +0000 (02:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=172945

Reviewed by JF Bastien.

The bug is a simple typo. It should use the constant
`true` instead of `false` when invoking the WASM_PARSER_FAIL_IF
macro. This failure is already caught by spec tests that fail
on arm64 devices.

* wasm/WasmModuleParser.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@217814 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/wasm/WasmModuleParser.cpp

index c1927d4..bb1da5b 100644 (file)
@@ -1,3 +1,17 @@
+2017-06-05  Saam Barati  <sbarati@apple.com>
+
+        We don't properly parse init_expr when the opcode is an unexpected opcode
+        https://bugs.webkit.org/show_bug.cgi?id=172945
+
+        Reviewed by JF Bastien.
+
+        The bug is a simple typo. It should use the constant
+        `true` instead of `false` when invoking the WASM_PARSER_FAIL_IF
+        macro. This failure is already caught by spec tests that fail
+        on arm64 devices.
+
+        * wasm/WasmModuleParser.cpp:
+
 2017-06-05  Keith Miller  <keith_miller@apple.com>
 
         OMG tier up checks should be a patchpoint
index 7d07500..60535f1 100644 (file)
@@ -534,7 +534,7 @@ auto ModuleParser::parseInitExpr(uint8_t& opcode, uint64_t& bitsOrImportNumber,
     }
 
     default:
-        WASM_PARSER_FAIL_IF(false, "unknown init_expr opcode ", opcode);
+        WASM_PARSER_FAIL_IF(true, "unknown init_expr opcode ", opcode);
     }
 
     uint8_t endOpcode;