From 4f362fc70323169de234f556cfae81bbeb5a257c Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Sun, 12 Sep 2021 23:10:16 +0100 Subject: [PATCH] LibJS: Set OrdinaryFunctionObject's realm from the Tnterpreter ...instead of relying on the VM having a current execution context. This was an incorrect assumption I made, and it caused onfoo attribute handler construction in LibWeb to crash. Just use the same mechanism as NativeFunction in the meantime. --- Userland/Libraries/LibJS/Runtime/OrdinaryFunctionObject.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/Libraries/LibJS/Runtime/OrdinaryFunctionObject.cpp b/Userland/Libraries/LibJS/Runtime/OrdinaryFunctionObject.cpp index b558040532..6d4fd0e5d0 100644 --- a/Userland/Libraries/LibJS/Runtime/OrdinaryFunctionObject.cpp +++ b/Userland/Libraries/LibJS/Runtime/OrdinaryFunctionObject.cpp @@ -43,7 +43,7 @@ OrdinaryFunctionObject::OrdinaryFunctionObject(GlobalObject& global_object, cons , m_body(body) , m_parameters(move(parameters)) , m_environment(parent_scope) - , m_realm(vm().current_realm()) + , m_realm(vm().interpreter_if_exists() ? &vm().interpreter().realm() : nullptr) , m_function_length(function_length) , m_kind(kind) , m_is_strict(is_strict)