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

Unified Diff: test/unittests/wasm/module-decoder-unittest.cc

Issue 2361053004: Revert of [wasm] Master CL for Binary 0xC changes. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 4 years, 3 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
Index: test/unittests/wasm/module-decoder-unittest.cc
diff --git a/test/unittests/wasm/module-decoder-unittest.cc b/test/unittests/wasm/module-decoder-unittest.cc
index 74157d57f7c096032bee5ad94ed3a4a346c4c23d..28ec527fbe39701a16d97c3f608a321b6bfeeeb5 100644
--- a/test/unittests/wasm/module-decoder-unittest.cc
+++ b/test/unittests/wasm/module-decoder-unittest.cc
@@ -14,16 +14,7 @@
namespace internal {
namespace wasm {
-#define WASM_INIT_EXPR_I32V_1(val) WASM_I32V_1(val), kExprEnd
-#define WASM_INIT_EXPR_I32V_2(val) WASM_I32V_2(val), kExprEnd
-#define WASM_INIT_EXPR_I32V_3(val) WASM_I32V_3(val), kExprEnd
-#define WASM_INIT_EXPR_I32V_4(val) WASM_I32V_4(val), kExprEnd
-#define WASM_INIT_EXPR_I32V_5(val) WASM_I32V_5(val), kExprEnd
-#define WASM_INIT_EXPR_F32(val) WASM_F32(val), kExprEnd
-#define WASM_INIT_EXPR_I64(val) WASM_I64(val), kExprEnd
-#define WASM_INIT_EXPR_F64(val) WASM_F64(val), kExprEnd
-#define WASM_INIT_EXPR_GLOBAL(index) WASM_GET_GLOBAL(index), kExprEnd
-
+#define EMPTY_FUNCTION(sig_index) 0, SIG_INDEX(sig_index), U16_LE(0)
#define SIZEOF_EMPTY_FUNCTION ((size_t)5)
#define EMPTY_BODY 0
#define SIZEOF_EMPTY_BODY ((size_t)1)
@@ -32,64 +23,64 @@
#define SIG_ENTRY_i_i SIG_ENTRY_x_x(kLocalI32, kLocalI32)
-#define UNKNOWN_SECTION(size) 0, U32V_1(size + 5), 4, 'l', 'u', 'l', 'z'
-
-#define SECTION(name, size) k##name##SectionCode, U32V_1(size)
+#define UNKNOWN_EMPTY_SECTION_NAME 1, '\0'
+#define UNKNOWN_SECTION_NAME 4, 'l', 'u', 'l', 'z'
+
+#define SECTION(NAME, EXTRA_SIZE) WASM_SECTION_##NAME, U32V_1(EXTRA_SIZE)
#define SIGNATURES_SECTION(count, ...) \
- SECTION(Type, 1 + 3 * (count)), U32V_1(count), __VA_ARGS__
+ SECTION(SIGNATURES, 1 + 3 * (count)), U32V_1(count), __VA_ARGS__
#define FUNCTION_SIGNATURES_SECTION(count, ...) \
- SECTION(Function, 1 + (count)), U32V_1(count), __VA_ARGS__
+ SECTION(FUNCTION_SIGNATURES, 1 + (count)), U32V_1(count), __VA_ARGS__
#define FOO_STRING 3, 'f', 'o', 'o'
#define NO_LOCAL_NAMES 0
-#define EMPTY_SIGNATURES_SECTION SECTION(Type, 1), 0
-#define EMPTY_FUNCTION_SIGNATURES_SECTION SECTION(Function, 1), 0
-#define EMPTY_FUNCTION_BODIES_SECTION SECTION(Code, 1), 0
-#define SECTION_NAMES(size) \
- kUnknownSectionCode, U32V_1(size + 5), 4, 'n', 'a', 'm', 'e'
-#define EMPTY_NAMES_SECTION SECTION_NAMES(1), 0
+#define EMPTY_SIGNATURES_SECTION SECTION(SIGNATURES, 1), 0
+#define EMPTY_FUNCTION_SIGNATURES_SECTION SECTION(FUNCTION_SIGNATURES, 1), 0
+#define EMPTY_FUNCTION_BODIES_SECTION SECTION(FUNCTION_BODIES, 1), 0
+#define EMPTY_NAMES_SECTION SECTION(NAMES, 1), 0
#define X1(...) __VA_ARGS__
#define X2(...) __VA_ARGS__, __VA_ARGS__
#define X3(...) __VA_ARGS__, __VA_ARGS__, __VA_ARGS__
#define X4(...) __VA_ARGS__, __VA_ARGS__, __VA_ARGS__, __VA_ARGS__
-#define ONE_EMPTY_FUNCTION SECTION(Function, 1 + 1 * 1), 1, X1(0)
-
-#define TWO_EMPTY_FUNCTIONS SECTION(Function, 1 + 2 * 1), 2, X2(0)
-
-#define THREE_EMPTY_FUNCTIONS SECTION(Function, 1 + 3 * 1), 3, X3(0)
-
-#define FOUR_EMPTY_FUNCTIONS SECTION(Function, 1 + 4 * 1), 4, X4(0)
-
-#define ONE_EMPTY_BODY \
- SECTION(Code, 1 + 1 * (1 + SIZEOF_EMPTY_BODY)) \
- , 1, X1(SIZEOF_EMPTY_BODY, EMPTY_BODY)
-
-#define TWO_EMPTY_BODIES \
- SECTION(Code, 1 + 2 * (1 + SIZEOF_EMPTY_BODY)) \
- , 2, X2(SIZEOF_EMPTY_BODY, EMPTY_BODY)
-
-#define THREE_EMPTY_BODIES \
- SECTION(Code, 1 + 3 * (1 + SIZEOF_EMPTY_BODY)) \
- , 3, X3(SIZEOF_EMPTY_BODY, EMPTY_BODY)
-
-#define FOUR_EMPTY_BODIES \
- SECTION(Code, 1 + 4 * (1 + SIZEOF_EMPTY_BODY)) \
- , 4, X4(SIZEOF_EMPTY_BODY, EMPTY_BODY)
+#define ONE_EMPTY_FUNCTION WASM_SECTION_FUNCTION_SIGNATURES, 1 + 1 * 1, 1, X1(0)
+
+#define TWO_EMPTY_FUNCTIONS \
+ WASM_SECTION_FUNCTION_SIGNATURES, 1 + 2 * 1, 2, X2(0)
+
+#define THREE_EMPTY_FUNCTIONS \
+ WASM_SECTION_FUNCTION_SIGNATURES, 1 + 3 * 1, 3, X3(0)
+
+#define FOUR_EMPTY_FUNCTIONS \
+ WASM_SECTION_FUNCTION_SIGNATURES, 1 + 4 * 1, 4, X4(0)
+
+#define ONE_EMPTY_BODY \
+ WASM_SECTION_FUNCTION_BODIES, 1 + 1 * (1 + SIZEOF_EMPTY_BODY), 1, \
+ X1(SIZEOF_EMPTY_BODY, EMPTY_BODY)
+
+#define TWO_EMPTY_BODIES \
+ WASM_SECTION_FUNCTION_BODIES, 1 + 2 * (1 + SIZEOF_EMPTY_BODY), 2, \
+ X2(SIZEOF_EMPTY_BODY, EMPTY_BODY)
+
+#define THREE_EMPTY_BODIES \
+ WASM_SECTION_FUNCTION_BODIES, 1 + 3 * (1 + SIZEOF_EMPTY_BODY), 3, \
+ X3(SIZEOF_EMPTY_BODY, EMPTY_BODY)
+
+#define FOUR_EMPTY_BODIES \
+ WASM_SECTION_FUNCTION_BODIES, 1 + 4 * (1 + SIZEOF_EMPTY_BODY), 4, \
+ X4(SIZEOF_EMPTY_BODY, EMPTY_BODY)
#define SIGNATURES_SECTION_VOID_VOID \
- SECTION(Type, 1 + SIZEOF_SIG_ENTRY_v_v), 1, SIG_ENTRY_v_v
-
-#define LINEAR_MEMORY_INDEX_0 0
-
-#define EXPECT_VERIFIES(data) \
- do { \
- ModuleResult result = DecodeModule(data, data + sizeof(data)); \
- EXPECT_TRUE(result.ok()); \
- if (result.val) delete result.val; \
+ SECTION(SIGNATURES, 1 + SIZEOF_SIG_ENTRY_v_v), 1, SIG_ENTRY_v_v
+
+#define EXPECT_VERIFIES(data) \
+ do { \
+ ModuleResult result = DecodeModule(data, data + arraysize(data)); \
+ EXPECT_TRUE(result.ok()); \
+ if (result.val) delete result.val; \
} while (false)
#define EXPECT_FAILURE_LEN(data, length) \
@@ -158,7 +149,8 @@
TEST_F(WasmModuleVerifyTest, WrongMagic) {
for (uint32_t x = 1; x; x <<= 1) {
- const byte data[] = {U32_LE(kWasmMagic ^ x), U32_LE(kWasmVersion)};
+ const byte data[] = {U32_LE(kWasmMagic ^ x), U32_LE(kWasmVersion),
+ SECTION(END, 0)};
ModuleResult result = DecodeModuleNoHeader(data, data + sizeof(data));
EXPECT_FALSE(result.ok());
if (result.val) delete result.val;
@@ -167,7 +159,8 @@
TEST_F(WasmModuleVerifyTest, WrongVersion) {
for (uint32_t x = 1; x; x <<= 1) {
- const byte data[] = {U32_LE(kWasmMagic), U32_LE(kWasmVersion ^ x)};
+ const byte data[] = {U32_LE(kWasmMagic), U32_LE(kWasmVersion ^ x),
+ SECTION(END, 0)};
ModuleResult result = DecodeModuleNoHeader(data, data + sizeof(data));
EXPECT_FALSE(result.ok());
if (result.val) delete result.val;
@@ -175,23 +168,23 @@
}
TEST_F(WasmModuleVerifyTest, DecodeEmpty) {
- ModuleResult result = DecodeModule(nullptr, 0);
- EXPECT_TRUE(result.ok());
- if (result.val) delete result.val;
+ static const byte data[] = {SECTION(END, 0)};
+ EXPECT_VERIFIES(data);
}
TEST_F(WasmModuleVerifyTest, OneGlobal) {
static const byte data[] = {
- SECTION(Global, 6), // --
- 1,
- kLocalI32, // local type
- 0, // immutable
- WASM_INIT_EXPR_I32V_1(13) // init
+ SECTION(GLOBALS, 5), // --
+ 1,
+ NAME_LENGTH(1),
+ 'g', // name
+ kLocalI32, // local type
+ 0, // exported
};
{
// Should decode to exactly one global.
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(1, result.val->globals.size());
EXPECT_EQ(0, result.val->functions.size());
@@ -199,11 +192,10 @@
const WasmGlobal* global = &result.val->globals.back();
+ EXPECT_EQ(1, global->name_length);
EXPECT_EQ(kAstI32, global->type);
EXPECT_EQ(0, global->offset);
- EXPECT_EQ(false, global->mutability);
- EXPECT_EQ(WasmInitExpr::kI32Const, global->init.kind);
- EXPECT_EQ(13, global->init.val.i32_const);
+ EXPECT_FALSE(global->exported);
if (result.val) delete result.val;
}
@@ -213,11 +205,12 @@
TEST_F(WasmModuleVerifyTest, Global_invalid_type) {
static const byte data[] = {
- SECTION(Global, 6), // --
- 1,
- 64, // invalid memory type
- 1, // mutable
- WASM_INIT_EXPR_I32V_1(33), // init
+ SECTION(GLOBALS, 5), // --
+ 1,
+ NAME_LENGTH(1),
+ 'g', // name
+ 64, // invalid memory type
+ 0, // exported
};
ModuleResult result = DecodeModule(data, data + sizeof(data));
@@ -227,11 +220,12 @@
TEST_F(WasmModuleVerifyTest, Global_invalid_type2) {
static const byte data[] = {
- SECTION(Global, 6), // --
- 1,
- kLocalVoid, // invalid memory type
- 1, // mutable
- WASM_INIT_EXPR_I32V_1(33), // init
+ SECTION(GLOBALS, 5), // --
+ 1,
+ NAME_LENGTH(1),
+ 'g', // name
+ kLocalVoid, // invalid memory type
+ 0, // exported
};
ModuleResult result = DecodeModule(data, data + sizeof(data));
@@ -241,10 +235,10 @@
TEST_F(WasmModuleVerifyTest, ZeroGlobals) {
static const byte data[] = {
- SECTION(Global, 1), // --
- 0, // declare 0 globals
- };
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ SECTION(GLOBALS, 1), // --
+ 0, // declare 0 globals
+ };
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
if (result.val) delete result.val;
}
@@ -265,15 +259,15 @@
TEST_F(WasmModuleVerifyTest, NGlobals) {
static const byte data[] = {
- kLocalF32, // memory type
- 0, // immutable
- WASM_INIT_EXPR_F32(7.7), // init
+ NO_NAME, // name length
+ kLocalF32, // memory type
+ 0, // exported
};
for (uint32_t i = 0; i < 1000000; i = i * 13 + 1) {
std::vector<byte> buffer;
size_t size = SizeOfVarInt(i) + i * sizeof(data);
- const byte globals[] = {kGlobalSectionCode, U32V_5(size)};
+ const byte globals[] = {WASM_SECTION_GLOBALS, U32V_5(size)};
for (size_t g = 0; g != sizeof(globals); ++g) {
buffer.push_back(globals[g]);
}
@@ -288,48 +282,74 @@
}
}
+TEST_F(WasmModuleVerifyTest, GlobalWithInvalidNameOffset) {
+ static const byte data[] = {
+ SECTION(GLOBALS, 7),
+ 1, // declare one global
+ NO_NAME, // name offset
+ 33, // memory type
+ 0, // exported
+ };
+
+ EXPECT_FAILURE(data);
+}
+
+TEST_F(WasmModuleVerifyTest, GlobalWithInvalidNameLength) {
+ static const byte data[] = {
+ SECTION(GLOBALS, 5), // --
+ 1,
+ NAME_LENGTH(56), // invalid length
+ 'g', // name
+ kLocalI32, // memory type
+ 0, // exported
+ };
+ EXPECT_FAILURE(data);
+}
+
TEST_F(WasmModuleVerifyTest, GlobalWithInvalidMemoryType) {
- static const byte data[] = {SECTION(Global, 7),
- 33, // memory type
- 0, // exported
- WASM_INIT_EXPR_I32V_1(1)};
+ static const byte data[] = {
+ SECTION(GLOBALS, 7),
+ 1, // declare one global
+ NO_NAME, // name offset
+ 33, // memory type
+ 0, // exported
+ };
EXPECT_FAILURE(data);
}
TEST_F(WasmModuleVerifyTest, TwoGlobals) {
static const byte data[] = {
- SECTION(Global, 21),
+ SECTION(GLOBALS, 7),
2,
+ NO_NAME, // #0: name length
kLocalF32, // type
- 0, // immutable
- WASM_INIT_EXPR_F32(22.0),
+ 0, // exported
+ NO_NAME, // #1: name length
kLocalF64, // type
- 1, // mutable
- WASM_INIT_EXPR_F64(23.0),
+ 1, // exported
};
{
// Should decode to exactly two globals.
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(2, result.val->globals.size());
EXPECT_EQ(0, result.val->functions.size());
EXPECT_EQ(0, result.val->data_segments.size());
const WasmGlobal* g0 = &result.val->globals[0];
-
+ const WasmGlobal* g1 = &result.val->globals[1];
+
+ EXPECT_EQ(0, g0->name_length);
EXPECT_EQ(kAstF32, g0->type);
EXPECT_EQ(0, g0->offset);
- EXPECT_EQ(false, g0->mutability);
- EXPECT_EQ(WasmInitExpr::kF32Const, g0->init.kind);
-
- const WasmGlobal* g1 = &result.val->globals[1];
-
+ EXPECT_FALSE(g0->exported);
+
+ EXPECT_EQ(0, g1->name_length);
EXPECT_EQ(kAstF64, g1->type);
EXPECT_EQ(8, g1->offset);
- EXPECT_EQ(true, g1->mutability);
- EXPECT_EQ(WasmInitExpr::kF64Const, g1->init.kind);
+ EXPECT_TRUE(g1->exported);
if (result.val) delete result.val;
}
@@ -344,23 +364,23 @@
}
{
- static const byte data[] = {SECTION(Type, 1 + SIZEOF_SIG_ENTRY_x_x), 1,
- SIG_ENTRY_i_i};
+ static const byte data[] = {SECTION(SIGNATURES, 1 + SIZEOF_SIG_ENTRY_x_x),
+ 1, SIG_ENTRY_i_i};
EXPECT_VERIFIES(data);
}
}
TEST_F(WasmModuleVerifyTest, MultipleSignatures) {
static const byte data[] = {
- SECTION(Type, 1 + SIZEOF_SIG_ENTRY_v_v + SIZEOF_SIG_ENTRY_x_x +
- SIZEOF_SIG_ENTRY_x_xx), // --
+ SECTION(SIGNATURES, 1 + SIZEOF_SIG_ENTRY_v_v + SIZEOF_SIG_ENTRY_x_x +
+ SIZEOF_SIG_ENTRY_x_xx), // --
3, // --
SIG_ENTRY_v_v, // void -> void
SIG_ENTRY_x_x(kLocalI32, kLocalF32), // f32 -> i32
SIG_ENTRY_x_xx(kLocalI32, kLocalF64, kLocalF64), // f64,f64 -> i32
};
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(3, result.val->signatures.size());
if (result.val->signatures.size() == 3) {
@@ -378,18 +398,16 @@
}
TEST_F(WasmModuleVerifyTest, OneDataSegment) {
- const byte kDataSegmentSourceOffset = 24;
+ const byte kDataSegmentSourceOffset = 30;
const byte data[] = {
- SECTION(Memory, 4),
- ENTRY_COUNT(1),
- kResizableMaximumFlag,
+ SECTION(MEMORY, 3),
28,
28,
- SECTION(Data, 11),
- ENTRY_COUNT(1),
- LINEAR_MEMORY_INDEX_0,
- WASM_INIT_EXPR_I32V_3(0x9bbaa), // dest addr
- U32V_1(3), // source size
+ 1,
+ SECTION(DATA_SEGMENTS, 8),
+ 1,
+ U32V_3(0x9bbaa), // dest addr
+ U32V_1(3), // source size
'a',
'b',
'c' // data bytes
@@ -397,7 +415,7 @@
{
EXPECT_VERIFIES(data);
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(0, result.val->globals.size());
EXPECT_EQ(0, result.val->functions.size());
@@ -405,39 +423,36 @@
const WasmDataSegment* segment = &result.val->data_segments.back();
- EXPECT_EQ(WasmInitExpr::kI32Const, segment->dest_addr.kind);
- EXPECT_EQ(0x9bbaa, segment->dest_addr.val.i32_const);
+ EXPECT_EQ(0x9bbaa, segment->dest_addr);
EXPECT_EQ(kDataSegmentSourceOffset, segment->source_offset);
EXPECT_EQ(3, segment->source_size);
+ EXPECT_TRUE(segment->init);
if (result.val) delete result.val;
}
- EXPECT_OFF_END_FAILURE(data, 14, sizeof(data));
+ EXPECT_OFF_END_FAILURE(data, 13, sizeof(data));
}
TEST_F(WasmModuleVerifyTest, TwoDataSegments) {
- const byte kDataSegment0SourceOffset = 24;
- const byte kDataSegment1SourceOffset = kDataSegment0SourceOffset + 11;
+ const byte kDataSegment0SourceOffset = 30;
+ const byte kDataSegment1SourceOffset = 30 + 8;
const byte data[] = {
- SECTION(Memory, 4),
- ENTRY_COUNT(1),
- kResizableMaximumFlag,
+ SECTION(MEMORY, 3),
28,
28,
- SECTION(Data, 29),
- ENTRY_COUNT(2), // segment count
- LINEAR_MEMORY_INDEX_0,
- WASM_INIT_EXPR_I32V_3(0x7ffee), // #0: dest addr
- U32V_1(4), // source size
+ 1,
+ SECTION(DATA_SEGMENTS, 23),
+ 2, // segment count
+ U32V_3(0x7ffee), // #0: dest addr
+ U32V_1(4), // source size
1,
2,
3,
- 4, // data bytes
- LINEAR_MEMORY_INDEX_0,
- WASM_INIT_EXPR_I32V_3(0x6ddcc), // #1: dest addr
- U32V_1(10), // source size
+ 4, // data bytes
+ U32V_3(0x6ddcc), // #1: dest addr
+ U32V_1(10), // source size
1,
2,
3,
@@ -451,7 +466,7 @@
};
{
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(0, result.val->globals.size());
EXPECT_EQ(0, result.val->functions.size());
@@ -460,79 +475,61 @@
const WasmDataSegment* s0 = &result.val->data_segments[0];
const WasmDataSegment* s1 = &result.val->data_segments[1];
- EXPECT_EQ(WasmInitExpr::kI32Const, s0->dest_addr.kind);
- EXPECT_EQ(0x7ffee, s0->dest_addr.val.i32_const);
+ EXPECT_EQ(0x7ffee, s0->dest_addr);
EXPECT_EQ(kDataSegment0SourceOffset, s0->source_offset);
EXPECT_EQ(4, s0->source_size);
-
- EXPECT_EQ(WasmInitExpr::kI32Const, s1->dest_addr.kind);
- EXPECT_EQ(0x6ddcc, s1->dest_addr.val.i32_const);
+ EXPECT_TRUE(s0->init);
+
+ EXPECT_EQ(0x6ddcc, s1->dest_addr);
EXPECT_EQ(kDataSegment1SourceOffset, s1->source_offset);
EXPECT_EQ(10, s1->source_size);
+ EXPECT_TRUE(s1->init);
if (result.val) delete result.val;
}
- EXPECT_OFF_END_FAILURE(data, 14, sizeof(data));
-}
-
-TEST_F(WasmModuleVerifyTest, DataSegment_wrong_init_type) {
- const byte data[] = {
- SECTION(Memory, 4),
- ENTRY_COUNT(1),
- kResizableMaximumFlag,
- 28,
- 28,
- SECTION(Data, 11),
- ENTRY_COUNT(1),
- LINEAR_MEMORY_INDEX_0,
- WASM_INIT_EXPR_F64(9.9), // dest addr
- U32V_1(3), // source size
- 'a',
- 'b',
- 'c' // data bytes
- };
-
- EXPECT_FAILURE(data);
+ EXPECT_OFF_END_FAILURE(data, 13, sizeof(data));
+}
+
+TEST_F(WasmModuleVerifyTest, DataSegmentWithInvalidDest) {
+ const int source_size = 3;
+
+ for (byte mem_pages = 1; mem_pages < 16; mem_pages++) {
+ int mem_size = mem_pages * 0x10000; // 64k pages.
+
+ for (int dest_addr = mem_size - source_size;
+ dest_addr < mem_size + source_size; dest_addr++) {
+ byte data[] = {SECTION(MEMORY, 3),
+ mem_pages,
+ mem_pages,
+ 1,
+ SECTION(DATA_SEGMENTS, 8),
+ 1,
+ U32V_3(dest_addr),
+ U32V_1(source_size),
+ 'a',
+ 'b',
+ 'c'};
+
+ if (dest_addr <= (mem_size - source_size)) {
+ EXPECT_VERIFIES(data);
+ } else {
+ EXPECT_FAILURE(data);
+ }
+ }
+ }
}
TEST_F(WasmModuleVerifyTest, OneIndirectFunction) {
static const byte data[] = {
- // sig#0 ---------------------------------------------------------------
+ // sig#0 -------------------------------------------------------
SIGNATURES_SECTION_VOID_VOID,
- // funcs ---------------------------------------------------------------
+ // funcs ------------------------------------------------------
ONE_EMPTY_FUNCTION,
- // table declaration ---------------------------------------------------
- SECTION(Table, 4), ENTRY_COUNT(1), kWasmAnyFunctionTypeForm, 0, 1};
-
- ModuleResult result = DecodeModule(data, data + sizeof(data));
- EXPECT_OK(result);
- if (result.ok()) {
- EXPECT_EQ(1, result.val->signatures.size());
- EXPECT_EQ(1, result.val->functions.size());
- EXPECT_EQ(1, result.val->function_tables.size());
- EXPECT_EQ(1, result.val->function_tables[0].values.size());
- EXPECT_EQ(-1, result.val->function_tables[0].values[0]);
- }
- if (result.val) delete result.val;
-}
-
-TEST_F(WasmModuleVerifyTest, OneIndirectFunction_one_entry) {
- static const byte data[] = {
- // sig#0 ---------------------------------------------------------------
- SIGNATURES_SECTION_VOID_VOID,
- // funcs ---------------------------------------------------------------
- ONE_EMPTY_FUNCTION,
- // table declaration ---------------------------------------------------
- SECTION(Table, 4), ENTRY_COUNT(1), kWasmAnyFunctionTypeForm, 0, 1,
- // elements ------------------------------------------------------------
- SECTION(Element, 7),
- 1, // entry count
- TABLE_INDEX(0), WASM_INIT_EXPR_I32V_1(0),
- 1, // elements count
- FUNC_INDEX(0)};
-
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ // indirect table ----------------------------------------------
+ SECTION(FUNCTION_TABLE, 2), 1, U32V_1(0)};
+
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
if (result.ok()) {
EXPECT_EQ(1, result.val->signatures.size());
@@ -547,30 +544,25 @@
TEST_F(WasmModuleVerifyTest, MultipleIndirectFunctions) {
static const byte data[] = {
// sig#0 -------------------------------------------------------
- SECTION(Type, 1 + SIZEOF_SIG_ENTRY_v_v + SIZEOF_SIG_ENTRY_v_x),
+ SECTION(SIGNATURES, 1 + SIZEOF_SIG_ENTRY_v_v + SIZEOF_SIG_ENTRY_v_x),
2, // --
SIG_ENTRY_v_v, // void -> void
SIG_ENTRY_v_x(kLocalI32), // void -> i32
// funcs ------------------------------------------------------
FOUR_EMPTY_FUNCTIONS,
- // table declaration -------------------------------------------
- SECTION(Table, 4), ENTRY_COUNT(1), kWasmAnyFunctionTypeForm, 0, 8,
- // table elements ----------------------------------------------
- SECTION(Element, 14),
- 1, // entry count
- TABLE_INDEX(0), WASM_INIT_EXPR_I32V_1(0),
- 8, // elements count
- FUNC_INDEX(0), // --
- FUNC_INDEX(1), // --
- FUNC_INDEX(2), // --
- FUNC_INDEX(3), // --
- FUNC_INDEX(0), // --
- FUNC_INDEX(1), // --
- FUNC_INDEX(2), // --
- FUNC_INDEX(3), // --
+ // indirect table ----------------------------------------------
+ SECTION(FUNCTION_TABLE, 9), 8,
+ U32V_1(0), // --
+ U32V_1(1), // --
+ U32V_1(2), // --
+ U32V_1(3), // --
+ U32V_1(0), // --
+ U32V_1(1), // --
+ U32V_1(2), // --
+ U32V_1(3), // --
FOUR_EMPTY_BODIES};
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
if (result.ok()) {
EXPECT_EQ(2, result.val->signatures.size());
@@ -589,7 +581,7 @@
// sig#0 -------------------------------------------------------
SIGNATURES_SECTION_VOID_VOID,
// indirect table ----------------------------------------------
- SECTION(Table, 4), ENTRY_COUNT(1), 1, 0, 0,
+ SECTION(FUNCTION_TABLE, 3), 1, 0, 0,
};
EXPECT_FAILURE(data);
@@ -602,7 +594,7 @@
// functions ---------------------------------------------------
ONE_EMPTY_FUNCTION,
// indirect table ----------------------------------------------
- SECTION(Table, 4), ENTRY_COUNT(1), 1, 1, 0,
+ SECTION(FUNCTION_TABLE, 3), 1, 1, 0,
};
EXPECT_FAILURE(data);
@@ -615,7 +607,7 @@
v8::internal::AccountingAllocator allocator;
Zone zone(&allocator);
FunctionSig* sig =
- DecodeWasmSignatureForTesting(&zone, data, data + sizeof(data));
+ DecodeWasmSignatureForTesting(&zone, data, data + arraysize(data));
EXPECT_TRUE(sig != nullptr);
EXPECT_EQ(0, sig->parameter_count());
@@ -627,7 +619,7 @@
LocalTypePair ret_type = kLocalTypes[i];
const byte data[] = {SIG_ENTRY_x(ret_type.code)};
FunctionSig* sig =
- DecodeWasmSignatureForTesting(zone(), data, data + sizeof(data));
+ DecodeWasmSignatureForTesting(zone(), data, data + arraysize(data));
EXPECT_TRUE(sig != nullptr);
EXPECT_EQ(0, sig->parameter_count());
@@ -641,7 +633,7 @@
LocalTypePair param_type = kLocalTypes[i];
const byte data[] = {SIG_ENTRY_v_x(param_type.code)};
FunctionSig* sig =
- DecodeWasmSignatureForTesting(zone(), data, data + sizeof(data));
+ DecodeWasmSignatureForTesting(zone(), data, data + arraysize(data));
EXPECT_TRUE(sig != nullptr);
EXPECT_EQ(1, sig->parameter_count());
@@ -657,7 +649,7 @@
LocalTypePair param_type = kLocalTypes[j];
const byte data[] = {SIG_ENTRY_x_x(ret_type.code, param_type.code)};
FunctionSig* sig =
- DecodeWasmSignatureForTesting(zone(), data, data + sizeof(data));
+ DecodeWasmSignatureForTesting(zone(), data, data + arraysize(data));
EXPECT_TRUE(sig != nullptr);
EXPECT_EQ(1, sig->parameter_count());
@@ -676,7 +668,7 @@
const byte data[] = {
SIG_ENTRY_x_xx(kLocalI32, p0_type.code, p1_type.code)};
FunctionSig* sig =
- DecodeWasmSignatureForTesting(zone(), data, data + sizeof(data));
+ DecodeWasmSignatureForTesting(zone(), data, data + arraysize(data));
EXPECT_TRUE(sig != nullptr);
EXPECT_EQ(2, sig->parameter_count());
@@ -707,7 +699,7 @@
byte data[] = {SIG_ENTRY_x_xx(kLocalI32, kLocalI32, kLocalI32)};
data[i] = kInvalidType;
FunctionSig* sig =
- DecodeWasmSignatureForTesting(zone(), data, data + sizeof(data));
+ DecodeWasmSignatureForTesting(zone(), data, data + arraysize(data));
EXPECT_EQ(nullptr, sig);
}
}
@@ -715,21 +707,21 @@
TEST_F(WasmSignatureDecodeTest, Fail_invalid_ret_type1) {
static const byte data[] = {SIG_ENTRY_x_x(kLocalVoid, kLocalI32)};
FunctionSig* sig =
- DecodeWasmSignatureForTesting(zone(), data, data + sizeof(data));
+ DecodeWasmSignatureForTesting(zone(), data, data + arraysize(data));
EXPECT_EQ(nullptr, sig);
}
TEST_F(WasmSignatureDecodeTest, Fail_invalid_param_type1) {
static const byte data[] = {SIG_ENTRY_x_x(kLocalI32, kLocalVoid)};
FunctionSig* sig =
- DecodeWasmSignatureForTesting(zone(), data, data + sizeof(data));
+ DecodeWasmSignatureForTesting(zone(), data, data + arraysize(data));
EXPECT_EQ(nullptr, sig);
}
TEST_F(WasmSignatureDecodeTest, Fail_invalid_param_type2) {
static const byte data[] = {SIG_ENTRY_x_xx(kLocalI32, kLocalI32, kLocalVoid)};
FunctionSig* sig =
- DecodeWasmSignatureForTesting(zone(), data, data + sizeof(data));
+ DecodeWasmSignatureForTesting(zone(), data, data + arraysize(data));
EXPECT_EQ(nullptr, sig);
}
@@ -750,8 +742,8 @@
kExprNop // body
};
- FunctionResult result =
- DecodeWasmFunction(isolate(), zone(), nullptr, data, data + sizeof(data));
+ FunctionResult result = DecodeWasmFunction(isolate(), zone(), nullptr, data,
+ data + arraysize(data));
EXPECT_OK(result);
if (result.val && result.ok()) {
@@ -760,7 +752,7 @@
EXPECT_EQ(0, function->sig->return_count());
EXPECT_EQ(0, function->name_offset);
EXPECT_EQ(SIZEOF_SIG_ENTRY_v_v, function->code_start_offset);
- EXPECT_EQ(sizeof(data), function->code_end_offset);
+ EXPECT_EQ(arraysize(data), function->code_end_offset);
// TODO(titzer): verify encoding of local declarations
}
@@ -774,7 +766,6 @@
TEST_F(WasmModuleVerifyTest, TheLoneliestOfValidModulesTheTrulyEmptyOne) {
const byte data[] = {
- 0, // unknown section code.
0, // Empty section name.
// No section name, no content, nothing but sadness.
0, // No section content.
@@ -784,14 +775,15 @@
TEST_F(WasmModuleVerifyTest, OnlyUnknownSectionEmpty) {
const byte data[] = {
- UNKNOWN_SECTION(0),
+ UNKNOWN_SECTION_NAME, 0,
};
EXPECT_VERIFIES(data);
}
TEST_F(WasmModuleVerifyTest, OnlyUnknownSectionNonEmpty) {
const byte data[] = {
- UNKNOWN_SECTION(5),
+ UNKNOWN_SECTION_NAME,
+ 5, // section size
0xff,
0xff,
0xff,
@@ -806,7 +798,9 @@
// signatures
SIGNATURES_SECTION_VOID_VOID,
// -----------------------------------------------------------
- UNKNOWN_SECTION(0)};
+ UNKNOWN_SECTION_NAME,
+ 0 // empty section
+ };
EXPECT_VERIFIES(data);
}
@@ -815,14 +809,29 @@
// signatures
SIGNATURES_SECTION_VOID_VOID,
// -----------------------------------------------------------
- UNKNOWN_SECTION(5), 0xff, 0xff, 0xff, 0xff, 0xff,
+ UNKNOWN_SECTION_NAME,
+ 5, // section size
+ 0xff, 0xff, 0xff, 0xff, 0xff,
+ };
+ EXPECT_VERIFIES(data);
+}
+
+TEST_F(WasmModuleVerifyTest, SignatureFollowedByUnknownSectionWithLongLEB) {
+ const byte data[] = {
+ // signatures
+ SIGNATURES_SECTION_VOID_VOID,
+ // -----------------------------------------------------------
+ UNKNOWN_SECTION_NAME, 0x81, 0x80, 0x80, 0x80,
+ 0x00, // section size: 1 but in a 5-byte LEB
+ 0,
};
EXPECT_VERIFIES(data);
}
TEST_F(WasmModuleVerifyTest, UnknownSectionOverflow) {
static const byte data[] = {
- UNKNOWN_SECTION(9),
+ UNKNOWN_EMPTY_SECTION_NAME,
+ 9, // section size
1,
2,
3,
@@ -839,7 +848,12 @@
TEST_F(WasmModuleVerifyTest, UnknownSectionUnderflow) {
static const byte data[] = {
- UNKNOWN_SECTION(333),
+ UNKNOWN_EMPTY_SECTION_NAME,
+ 0xff,
+ 0xff,
+ 0xff,
+ 0xff,
+ 0x0f, // Section size LEB128 0xffffffff
1,
2,
3,
@@ -848,17 +862,35 @@
EXPECT_FAILURE(data);
}
+TEST_F(WasmModuleVerifyTest, UnknownSectionLoop) {
+ // Would infinite loop decoding if wrapping and allowed.
+ static const byte data[] = {
+ UNKNOWN_EMPTY_SECTION_NAME,
+ 1,
+ 2,
+ 3,
+ 4, // 4 byte section
+ 0xfa,
+ 0xff,
+ 0xff,
+ 0xff,
+ 0x0f, // Section size LEB128 0xfffffffa
+ };
+ EXPECT_FAILURE(data);
+}
+
TEST_F(WasmModuleVerifyTest, UnknownSectionSkipped) {
static const byte data[] = {
- UNKNOWN_SECTION(1),
+ UNKNOWN_EMPTY_SECTION_NAME,
+ 1, // section size
0, // one byte section
- SECTION(Global, 6),
- 1,
- kLocalI32, // memory type
- 0, // exported
- WASM_INIT_EXPR_I32V_1(33), // init
- };
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ SECTION(GLOBALS, 4),
+ 1,
+ 0, // name length
+ kLocalI32, // memory type
+ 0, // exported
+ };
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(1, result.val->globals.size());
@@ -867,41 +899,42 @@
const WasmGlobal* global = &result.val->globals.back();
+ EXPECT_EQ(0, global->name_length);
EXPECT_EQ(kAstI32, global->type);
EXPECT_EQ(0, global->offset);
+ EXPECT_FALSE(global->exported);
if (result.val) delete result.val;
}
TEST_F(WasmModuleVerifyTest, ImportTable_empty) {
- static const byte data[] = {SECTION(Type, 1), 0, SECTION(Import, 1), 0};
+ static const byte data[] = {SECTION(SIGNATURES, 1), 0,
+ SECTION(IMPORT_TABLE, 1), 0};
EXPECT_VERIFIES(data);
}
TEST_F(WasmModuleVerifyTest, ImportTable_nosigs1) {
- static const byte data[] = {SECTION(Import, 1), 0};
+ static const byte data[] = {SECTION(IMPORT_TABLE, 1), 0};
EXPECT_VERIFIES(data);
}
TEST_F(WasmModuleVerifyTest, ImportTable_nosigs2) {
static const byte data[] = {
- SECTION(Import, 6), 1, // sig table
- NAME_LENGTH(1), 'm', // module name
- NAME_LENGTH(1), 'f', // function name
- kExternalFunction, // import kind
- IMPORT_SIG_INDEX(0), // sig index
+ SECTION(IMPORT_TABLE, 6), 1, // sig table
+ IMPORT_SIG_INDEX(0), // sig index
+ NAME_LENGTH(1), 'm', // module name
+ NAME_LENGTH(1), 'f', // function name
};
EXPECT_FAILURE(data);
}
TEST_F(WasmModuleVerifyTest, ImportTable_invalid_sig) {
static const byte data[] = {
- SECTION(Type, 1), 0, // --
- SECTION(Import, 6), 1, // --
- NAME_LENGTH(1), 'm', // module name
- NAME_LENGTH(1), 'f', // function name
- kExternalFunction, // import kind
- IMPORT_SIG_INDEX(0), // sig index
+ SECTION(SIGNATURES, 1), 0, // --
+ SECTION(IMPORT_TABLE, 6), 1, // --
+ IMPORT_SIG_INDEX(0), // sig index
+ NAME_LENGTH(1), 'm', // module name
+ NAME_LENGTH(1), 'f', // function name
};
EXPECT_FAILURE(data);
}
@@ -910,58 +943,55 @@
static const byte data[] = {
// signatures
SIGNATURES_SECTION_VOID_VOID,
- SECTION(Import, 7),
- 1, // --
+ SECTION(IMPORT_TABLE, 6),
+ 1, // --
+ IMPORT_SIG_INDEX(0), // sig index
NAME_LENGTH(1),
'm', // module name
NAME_LENGTH(1),
- 'f', // function name
- kExternalFunction, // import kind
- IMPORT_SIG_INDEX(0), // sig index
+ 'f', // function name
};
EXPECT_VERIFIES(data);
}
TEST_F(WasmModuleVerifyTest, ImportTable_invalid_module) {
- static const byte data[] = {
- // signatures
- SIGNATURES_SECTION_VOID_VOID, // --
- SECTION(Import, 7), // --
- 1, // --
- NO_NAME, // module name
- NAME_LENGTH(1), // --
- 'f', // function name
- kExternalFunction, // import kind
- IMPORT_SIG_INDEX(0), // sig index
- };
- EXPECT_FAILURE(data);
-}
-
-TEST_F(WasmModuleVerifyTest, ImportTable_off_end) {
static const byte data[] = {
// signatures
SIGNATURES_SECTION_VOID_VOID,
- SECTION(Import, 6),
- 1,
+ SECTION(IMPORT_TABLE, 6),
+ 1, // --
+ IMPORT_SIG_INDEX(0), // sig index
+ NO_NAME, // module name
+ NAME_LENGTH(1),
+ 'f' // function name
+ };
+ EXPECT_FAILURE(data);
+}
+
+TEST_F(WasmModuleVerifyTest, ImportTable_off_end) {
+ static const byte data[] = {
+ // signatures
+ SIGNATURES_SECTION_VOID_VOID,
+ SECTION(IMPORT_TABLE, 6),
+ 1,
+ IMPORT_SIG_INDEX(0), // sig index
NAME_LENGTH(1),
'm', // module name
NAME_LENGTH(1),
- 'f', // function name
- kExternalFunction, // import kind
- IMPORT_SIG_INDEX(0), // sig index
+ 'f', // function name
};
EXPECT_OFF_END_FAILURE(data, 16, sizeof(data));
}
TEST_F(WasmModuleVerifyTest, ExportTable_empty1) {
- static const byte data[] = { // signatures
- SIGNATURES_SECTION_VOID_VOID, // --
- ONE_EMPTY_FUNCTION, SECTION(Export, 1), // --
- 0, // --
+ static const byte data[] = {// signatures
+ SIGNATURES_SECTION_VOID_VOID, ONE_EMPTY_FUNCTION,
+ SECTION(EXPORT_TABLE, 1),
+ 0, // --
ONE_EMPTY_BODY};
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(1, result.val->functions.size());
@@ -972,27 +1002,34 @@
TEST_F(WasmModuleVerifyTest, ExportTable_empty2) {
static const byte data[] = {
- SECTION(Type, 1), 0, SECTION(Export, 1), 0 // --
- };
- EXPECT_VERIFIES(data);
+ SECTION(SIGNATURES, 1), 0, SECTION(EXPORT_TABLE, 1), 0 // --
+ };
+ // TODO(titzer): current behavior treats empty functions section as missing.
+ EXPECT_FAILURE(data);
+}
+
+TEST_F(WasmModuleVerifyTest, ExportTable_NoFunctions1) {
+ static const byte data[] = {
+ SECTION(SIGNATURES, 1), 0, SECTION(EXPORT_TABLE, 1), 0 // --
+ };
+ EXPECT_FAILURE(data);
}
TEST_F(WasmModuleVerifyTest, ExportTable_NoFunctions2) {
- static const byte data[] = {SECTION(Export, 1), 0};
- EXPECT_VERIFIES(data);
+ static const byte data[] = {SECTION(EXPORT_TABLE, 1), 0};
+ EXPECT_FAILURE(data);
}
TEST_F(WasmModuleVerifyTest, ExportTableOne) {
static const byte data[] = {// signatures
SIGNATURES_SECTION_VOID_VOID,
ONE_EMPTY_FUNCTION,
- SECTION(Export, 4),
- 1, // exports
- NO_NAME, // --
- kExternalFunction, // --
- FUNC_INDEX(0), // --
+ SECTION(EXPORT_TABLE, 3),
+ 1, // exports
+ FUNC_INDEX(0), // --
+ NO_NAME, // --
ONE_EMPTY_BODY};
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(1, result.val->functions.size());
@@ -1005,12 +1042,12 @@
static const byte data[] = {// signatures
SIGNATURES_SECTION_VOID_VOID,
ONE_EMPTY_FUNCTION,
- SECTION(Export, 12),
- 1, // exports
- NAME_LENGTH(84), // invalid string length
- 'e', // --
- kExternalFunction, // --
- FUNC_INDEX(0)};
+ SECTION(EXPORT_TABLE, 12),
+ 1, // exports
+ FUNC_INDEX(0), // --
+ NAME_LENGTH(84), // invalid string length
+ 'e', // --
+ ONE_EMPTY_BODY};
EXPECT_FAILURE(data);
}
@@ -1019,24 +1056,22 @@
static const byte data[] = {// signatures
SIGNATURES_SECTION_VOID_VOID,
ONE_EMPTY_FUNCTION,
- SECTION(Export, 14),
- 2, // exports
+ SECTION(EXPORT_TABLE, 12),
+ 2, // exports
+ FUNC_INDEX(0), // --
NAME_LENGTH(4),
'n',
'a',
'm',
- 'e', // --
- kExternalFunction,
+ 'e', // --
FUNC_INDEX(0), // --
NAME_LENGTH(3),
'n',
'o',
- 'm', // --
- kExternalFunction, // --
- FUNC_INDEX(0), // --
+ 'm', // --
ONE_EMPTY_BODY};
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(1, result.val->functions.size());
@@ -1049,22 +1084,19 @@
static const byte data[] = {// signatures
SIGNATURES_SECTION_VOID_VOID,
THREE_EMPTY_FUNCTIONS,
- SECTION(Export, 13),
- 3, // exports
- NAME_LENGTH(1),
- 'a', // --
- kExternalFunction,
+ SECTION(EXPORT_TABLE, 10),
+ 3, // exports
FUNC_INDEX(0), // --
NAME_LENGTH(1),
- 'b', // --
- kExternalFunction,
+ 'a', // --
FUNC_INDEX(1), // --
NAME_LENGTH(1),
+ 'b', // --
+ FUNC_INDEX(2), // --
+ NAME_LENGTH(1),
'c', // --
- kExternalFunction,
- FUNC_INDEX(2), // --
THREE_EMPTY_BODIES};
- ModuleResult result = DecodeModule(data, data + sizeof(data));
+ ModuleResult result = DecodeModule(data, data + arraysize(data));
EXPECT_OK(result);
EXPECT_EQ(3, result.val->functions.size());
@@ -1078,13 +1110,12 @@
const byte data[] = {// signatures
SIGNATURES_SECTION_VOID_VOID,
THREE_EMPTY_FUNCTIONS,
- SECTION(Export, 6),
- 1, // exports
+ SECTION(EXPORT_TABLE, 5),
+ 1, // exports
+ FUNC_INDEX(i), // --
NAME_LENGTH(2),
'e',
'x', // --
- kExternalFunction,
- FUNC_INDEX(i), // --
THREE_EMPTY_BODIES};
if (i < 3) {
@@ -1100,11 +1131,10 @@
// signatures
SIGNATURES_SECTION_VOID_VOID,
ONE_EMPTY_FUNCTION,
- SECTION(Export, 1 + 6),
- 1, // exports
- NO_NAME, // --
- kExternalFunction,
+ SECTION(EXPORT_TABLE, 1 + 6),
+ 1, // exports
FUNC_INDEX(0), // --
+ NO_NAME // --
};
for (int length = 33; length < sizeof(data); length++) {
@@ -1116,9 +1146,9 @@
TEST_F(WasmModuleVerifyTest, FunctionSignatures_empty) {
static const byte data[] = {
- SECTION(Type, 1), 0, // --
- SECTION(Function, 1), 0 // --
- }; // --
+ SECTION(SIGNATURES, 1), 0, // --
+ SECTION(FUNCTION_SIGNATURES, 1), 0 // --
+ }; // --
EXPECT_VERIFIES(data);
}
@@ -1132,10 +1162,10 @@
TEST_F(WasmModuleVerifyTest, Regression_648070) {
static const byte data[] = {
- SECTION(Type, 1), 0, // --
- SECTION(Function, 5), // --
- U32V_5(3500228624) // function count = 3500228624
- }; // --
+ SECTION(SIGNATURES, 1), 0, // --
+ SECTION(FUNCTION_SIGNATURES, 5), // --
+ U32V_5(3500228624) // function count = 3500228624
+ }; // --
EXPECT_FAILURE(data);
}
@@ -1150,40 +1180,40 @@
TEST_F(WasmModuleVerifyTest, FunctionBodies_one_empty) {
static const byte data[] = {
+ SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
+ FUNCTION_SIGNATURES_SECTION(1, 0), // --
+ SECTION(FUNCTION_BODIES, 1 + SIZEOF_EMPTY_BODY), 1, EMPTY_BODY // --
+ };
+ EXPECT_VERIFIES(data);
+}
+
+TEST_F(WasmModuleVerifyTest, FunctionBodies_one_nop) {
+ static const byte data[] = {
+ SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
+ FUNCTION_SIGNATURES_SECTION(1, 0), // --
+ SECTION(FUNCTION_BODIES, 1 + SIZEOF_NOP_BODY), 1, NOP_BODY // --
+ };
+ EXPECT_VERIFIES(data);
+}
+
+TEST_F(WasmModuleVerifyTest, FunctionBodies_count_mismatch1) {
+ static const byte data[] = {
SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
- FUNCTION_SIGNATURES_SECTION(1, 0), // --
- SECTION(Code, 1 + SIZEOF_EMPTY_BODY), 1, EMPTY_BODY // --
- };
- EXPECT_VERIFIES(data);
-}
-
-TEST_F(WasmModuleVerifyTest, FunctionBodies_one_nop) {
- static const byte data[] = {
- SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
- FUNCTION_SIGNATURES_SECTION(1, 0), // --
- SECTION(Code, 1 + SIZEOF_NOP_BODY), 1, NOP_BODY // --
- };
- EXPECT_VERIFIES(data);
-}
-
-TEST_F(WasmModuleVerifyTest, FunctionBodies_count_mismatch1) {
- static const byte data[] = {
- SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
- FUNCTION_SIGNATURES_SECTION(2, 0, 0), // --
- SECTION(Code, 1 + SIZEOF_EMPTY_BODY), 1, // --
- EMPTY_BODY // --
+ FUNCTION_SIGNATURES_SECTION(2, 0, 0), // --
+ SECTION(FUNCTION_BODIES, 1 + SIZEOF_EMPTY_BODY), 1, // --
+ EMPTY_BODY // --
};
EXPECT_FAILURE(data);
}
TEST_F(WasmModuleVerifyTest, FunctionBodies_count_mismatch2) {
static const byte data[] = {
- SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
- FUNCTION_SIGNATURES_SECTION(1, 0), // --
- SECTION(Code, 1 + 2 * SIZEOF_NOP_BODY), // --
- ENTRY_COUNT(2), // --
- NOP_BODY, // --
- NOP_BODY // --
+ SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
+ FUNCTION_SIGNATURES_SECTION(1, 0), // --
+ SECTION(FUNCTION_BODIES, 1 + 2 * SIZEOF_NOP_BODY), // --
+ 2, // --
+ NOP_BODY, // --
+ NOP_BODY // --
};
EXPECT_FAILURE(data);
}
@@ -1199,11 +1229,11 @@
static const byte data[] = {
SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
FUNCTION_SIGNATURES_SECTION(1, 0), // --
- SECTION(Code, 1 + SIZEOF_EMPTY_BODY),
- ENTRY_COUNT(1),
+ SECTION(FUNCTION_BODIES, 1 + SIZEOF_EMPTY_BODY),
+ 1,
EMPTY_BODY, // --
- SECTION_NAMES(1 + 5),
- ENTRY_COUNT(1),
+ SECTION(NAMES, 1 + 5),
+ 1,
FOO_STRING,
NO_LOCAL_NAMES // --
};
@@ -1212,14 +1242,14 @@
TEST_F(WasmModuleVerifyTest, Names_two_empty) {
static const byte data[] = {
- SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
- FUNCTION_SIGNATURES_SECTION(2, 0, 0), // --
- SECTION(Code, 1 + 2 * SIZEOF_EMPTY_BODY), // --
- ENTRY_COUNT(2),
+ SIGNATURES_SECTION(1, SIG_ENTRY_v_v), // --
+ FUNCTION_SIGNATURES_SECTION(2, 0, 0), // --
+ SECTION(FUNCTION_BODIES, 1 + 2 * SIZEOF_EMPTY_BODY), // --
+ 2,
EMPTY_BODY,
EMPTY_BODY, // --
- SECTION_NAMES(1 + 10),
- ENTRY_COUNT(2), // --
+ SECTION(NAMES, 1 + 10),
+ 2, // --
FOO_STRING,
NO_LOCAL_NAMES, // --
FOO_STRING,
@@ -1228,66 +1258,6 @@
EXPECT_VERIFIES(data);
}
-#define EXPECT_INIT_EXPR(Type, type, value, ...) \
- { \
- static const byte data[] = {__VA_ARGS__, kExprEnd}; \
- WasmInitExpr expr = \
- DecodeWasmInitExprForTesting(data, data + sizeof(data)); \
- EXPECT_EQ(WasmInitExpr::k##Type##Const, expr.kind); \
- EXPECT_EQ(value, expr.val.type##_const); \
- }
-
-TEST_F(WasmModuleVerifyTest, InitExpr_i32) {
- EXPECT_INIT_EXPR(I32, i32, 33, WASM_I32V_1(33));
- EXPECT_INIT_EXPR(I32, i32, -21, WASM_I32V_1(-21));
- EXPECT_INIT_EXPR(I32, i32, 437, WASM_I32V_2(437));
- EXPECT_INIT_EXPR(I32, i32, 77777, WASM_I32V_3(77777));
-}
-
-TEST_F(WasmModuleVerifyTest, InitExpr_f32) {
- EXPECT_INIT_EXPR(F32, f32, static_cast<float>(13.1), WASM_F32(13.1));
- EXPECT_INIT_EXPR(F32, f32, static_cast<float>(-21.1), WASM_F32(-21.1));
- EXPECT_INIT_EXPR(F32, f32, static_cast<float>(437.2), WASM_F32(437.2));
- EXPECT_INIT_EXPR(F32, f32, static_cast<float>(77777.3), WASM_F32(77777.3));
-}
-
-TEST_F(WasmModuleVerifyTest, InitExpr_i64) {
- EXPECT_INIT_EXPR(I64, i64, 33, WASM_I64V_1(33));
- EXPECT_INIT_EXPR(I64, i64, -21, WASM_I64V_2(-21));
- EXPECT_INIT_EXPR(I64, i64, 437, WASM_I64V_5(437));
- EXPECT_INIT_EXPR(I64, i64, 77777, WASM_I64V_7(77777));
-}
-
-TEST_F(WasmModuleVerifyTest, InitExpr_f64) {
- EXPECT_INIT_EXPR(F64, f64, 83.22, WASM_F64(83.22));
- EXPECT_INIT_EXPR(F64, f64, -771.3, WASM_F64(-771.3));
- EXPECT_INIT_EXPR(F64, f64, 43703.0, WASM_F64(43703.0));
- EXPECT_INIT_EXPR(F64, f64, 77999.1, WASM_F64(77999.1));
-}
-
-#define EXPECT_INIT_EXPR_FAIL(...) \
- { \
- static const byte data[] = {__VA_ARGS__, kExprEnd}; \
- WasmInitExpr expr = \
- DecodeWasmInitExprForTesting(data, data + sizeof(data)); \
- EXPECT_EQ(WasmInitExpr::kNone, expr.kind); \
- }
-
-TEST_F(WasmModuleVerifyTest, InitExpr_illegal) {
- EXPECT_INIT_EXPR_FAIL(WASM_I32V_1(0), WASM_I32V_1(0));
- EXPECT_INIT_EXPR_FAIL(WASM_GET_LOCAL(0));
- EXPECT_INIT_EXPR_FAIL(WASM_SET_LOCAL(0, WASM_I32V_1(0)));
- EXPECT_INIT_EXPR_FAIL(WASM_I32_ADD(WASM_I32V_1(0), WASM_I32V_1(0)));
- EXPECT_INIT_EXPR_FAIL(WASM_IF_ELSE(WASM_ZERO, WASM_ZERO, WASM_ZERO));
-}
-
-TEST_F(WasmModuleVerifyTest, InitExpr_global) {
- static const byte data[] = {WASM_INIT_EXPR_GLOBAL(37)};
- WasmInitExpr expr = DecodeWasmInitExprForTesting(data, data + sizeof(data));
- EXPECT_EQ(WasmInitExpr::kGlobalIndex, expr.kind);
- EXPECT_EQ(37, expr.val.global_index);
-}
-
} // namespace wasm
} // namespace internal
} // namespace v8
« no previous file with comments | « test/unittests/wasm/loop-assignment-analysis-unittest.cc ('k') | test/unittests/wasm/wasm-macro-gen-unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698