mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-06-24 19:21:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibWasm: Ensure that global.get only accesses imports in const exprs

Browse source
This commit is contained in:
Diego 2024-06-16 09:55:51 -07:00 • committed by Ali Mohammad Pur
parent 596dd5252d
commit bd97091cbb
Notes: sideshowbarker 2024-07-17 20:19:08 +09:00
2 changed files with 8 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

8
Userland/Libraries/LibWasm/AbstractMachine/AbstractMachine.cpp
View file

@ -206,6 +206,8 @@ InstantiationResult AbstractMachine::instantiate(Module const& module, Vector<Ex
for (auto& entry : externs) { for (auto& entry : externs) {
if (auto* ptr = entry.get_pointer<GlobalAddress>()) if (auto* ptr = entry.get_pointer<GlobalAddress>())
auxiliary_instance.globals().append(*ptr); auxiliary_instance.globals().append(*ptr);
else if (auto* ptr = entry.get_pointer<FunctionAddress>())
auxiliary_instance.functions().append(*ptr);
} }
Vector<FunctionAddress> module_functions; Vector<FunctionAddress> module_functions;
@ -253,7 +255,7 @@ InstantiationResult AbstractMachine::instantiate(Module const& module, Vector<Ex
if (m_should_limit_instruction_count) if (m_should_limit_instruction_count)
config.enable_instruction_count_limit(); config.enable_instruction_count_limit();
config.set_frame(Frame { config.set_frame(Frame {
main_module_instance, auxiliary_instance,
Vector<Value> {}, Vector<Value> {},
entry, entry,
entry.instructions().size(), entry.instructions().size(),
@ -306,7 +308,7 @@ InstantiationResult AbstractMachine::instantiate(Module const& module, Vector<Ex
if (m_should_limit_instruction_count) if (m_should_limit_instruction_count)
config.enable_instruction_count_limit(); config.enable_instruction_count_limit();
config.set_frame(Frame { config.set_frame(Frame {
main_module_instance, auxiliary_instance,
Vector<Value> {}, Vector<Value> {},
active_ptr->expression, active_ptr->expression,
1, 1,
@ -361,7 +363,7 @@ InstantiationResult AbstractMachine::instantiate(Module const& module, Vector<Ex
if (m_should_limit_instruction_count) if (m_should_limit_instruction_count)
config.enable_instruction_count_limit(); config.enable_instruction_count_limit();
config.set_frame(Frame { config.set_frame(Frame {
main_module_instance, auxiliary_instance,
Vector<Value> {}, Vector<Value> {},
data.offset, data.offset,
1, 1,

3
Userland/Libraries/LibWasm/AbstractMachine/BytecodeInterpreter.cpp
View file

@ -728,6 +728,9 @@ void BytecodeInterpreter::interpret(Configuration& configuration, InstructionPoi
} }
case Instructions::global_get.value(): { case Instructions::global_get.value(): {
auto global_index = instruction.arguments().get<GlobalIndex>(); auto global_index = instruction.arguments().get<GlobalIndex>();
// This check here is for const expressions. In non-const expressions,
// a validation error would have been thrown.
TRAP_IF_NOT(global_index < configuration.frame().module().globals().size());
auto address = configuration.frame().module().globals()[global_index.value()]; auto address = configuration.frame().module().globals()[global_index.value()];
dbgln_if(WASM_TRACE_DEBUG, "global({}) -> stack", address.value()); dbgln_if(WASM_TRACE_DEBUG, "global({}) -> stack", address.value());
auto global = configuration.store().get(address); auto global = configuration.store().get(address);