fergalmoran/ladybird
1
0
Fork 0
mirror of https://github.com/fergalmoran/ladybird.git synced 2025-12-22 09:19:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

LibWeb: Always assert that principal realm returns a principal realm

Browse Source 
There was a bug in the HTML proposal where a synthetic realm settings
object's principal realm was a shadow realm if there were nested shadow
realms, which this assertion catches more directly (rather than later
down the track, where it is used).

We were meant to also assert for this case, but we were previously
returning early.
This commit is contained in:
Shannon Booth
2024-11-28 02:49:34 +13:00
committed by Andreas Kling
parent 27f1e3676f
commit bb10b0e301
2 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
Libraries/LibWeb/HTML/Scripting/Environments.cpp
View File

@@ -360,19 +360,19 @@ JS::Realm& current_principal_realm()
}
// https://whatpr.org/html/9893/webappapis.html#concept-principal-realm-of-realm
JS::Realm& principal_realm(JS::Realm& realm)
JS::Realm& principal_realm(GC::Ref<JS::Realm> realm)
{
VERIFY(realm.host_defined());
VERIFY(realm->host_defined());
// 1. If realm.[[HostDefined]] is a synthetic realm settings object, then:
if (is<Bindings::SyntheticHostDefined>(*realm.host_defined())) {
if (is<Bindings::SyntheticHostDefined>(*realm->host_defined())) {
// 1. Assert: realm is a synthetic realm.
// 2. Set realm to the principal realm of realm.[[HostDefined]].
return static_cast<Bindings::SyntheticHostDefined const&>(*realm.host_defined()).synthetic_realm_settings.principal_realm;
realm = static_cast<Bindings::SyntheticHostDefined const&>(*realm->host_defined()).synthetic_realm_settings.principal_realm;
}
// 2. Assert: realm.[[HostDefined]] is an environment settings object and realm is a principal realm.
VERIFY(is<Bindings::PrincipalHostDefined>(*realm.host_defined()));
VERIFY(is<Bindings::PrincipalHostDefined>(*realm->host_defined()));
// 3. Return realm.
return realm;