summaryrefslogtreecommitdiff
path: root/Userland/Libraries/LibWasm
diff options
context:
space:
mode:
Diffstat (limited to 'Userland/Libraries/LibWasm')
-rw-r--r--Userland/Libraries/LibWasm/AbstractMachine/Configuration.cpp3
-rw-r--r--Userland/Libraries/LibWasm/AbstractMachine/Interpreter.cpp15
2 files changed, 16 insertions, 2 deletions
diff --git a/Userland/Libraries/LibWasm/AbstractMachine/Configuration.cpp b/Userland/Libraries/LibWasm/AbstractMachine/Configuration.cpp
index fcfd14ddee..ac93f77507 100644
--- a/Userland/Libraries/LibWasm/AbstractMachine/Configuration.cpp
+++ b/Userland/Libraries/LibWasm/AbstractMachine/Configuration.cpp
@@ -66,6 +66,9 @@ Result Configuration::execute(Interpreter& interpreter)
if (interpreter.did_trap())
return Trap {};
+ if (stack().size() <= frame().arity() + 1)
+ return Trap {};
+
Vector<Value> results;
results.ensure_capacity(frame().arity());
for (size_t i = 0; i < frame().arity(); ++i)
diff --git a/Userland/Libraries/LibWasm/AbstractMachine/Interpreter.cpp b/Userland/Libraries/LibWasm/AbstractMachine/Interpreter.cpp
index afebe0ef97..6e625c7aa8 100644
--- a/Userland/Libraries/LibWasm/AbstractMachine/Interpreter.cpp
+++ b/Userland/Libraries/LibWasm/AbstractMachine/Interpreter.cpp
@@ -122,6 +122,7 @@ void BytecodeInterpreter::call_address(Configuration& configuration, FunctionAdd
const FunctionType* type { nullptr };
instance->visit([&](const auto& function) { type = &function.type(); });
TRAP_IF_NOT(type);
+ TRAP_IF_NOT(configuration.stack().entries().size() > type->parameters().size());
Vector<Value> args;
args.ensure_capacity(type->parameters().size());
auto span = configuration.stack().entries().span().slice_from_end(type->parameters().size());
@@ -506,8 +507,18 @@ void BytecodeInterpreter::interpret(Configuration& configuration, InstructionPoi
return;
return branch_to_label(configuration, instruction.arguments().get<LabelIndex>());
}
- case Instructions::br_table.value():
- goto unimplemented;
+ case Instructions::br_table.value(): {
+ auto& arguments = instruction.arguments().get<Instruction::TableBranchArgs>();
+ auto entry = configuration.stack().pop();
+ TRAP_IF_NOT(entry.has<Value>());
+ auto maybe_i = entry.get<Value>().to<i32>();
+ TRAP_IF_NOT(maybe_i.has_value());
+ TRAP_IF_NOT(maybe_i.value() >= 0);
+ size_t i = *maybe_i;
+ if (i < arguments.labels.size())
+ return branch_to_label(configuration, arguments.labels[i]);
+ return branch_to_label(configuration, arguments.default_);
+ }
case Instructions::call.value(): {
auto index = instruction.arguments().get<FunctionIndex>();
TRAP_IF_NOT(index.value() < configuration.frame().module().functions().size());