Medium firefox Logic Error 🔧 Commit mapped

Overview

Medium
Severity
CVSS
No
Exploited ITW
Fixed
Fix Status
Impactmoderate
DescriptionSite isolation issue in the DOM: Navigation component
ComponentDOM
Bug ClassLogic Error
Tracker2049981
Fix commitcd3ea2c86c26 (firefox) +5/-1
CISA KEVNot listed
CreditedYaqoub Aldurayhim
Disclosed2026-07-21

Files Changed

  • dom/ipc/BrowserParent.cpp
diff --git a/dom/ipc/BrowserParent.cpp b/dom/ipc/BrowserParent.cpp
index 6a6126b3640..ca69f14836c 100644
--- a/dom/ipc/BrowserParent.cpp
+++ b/dom/ipc/BrowserParent.cpp
@@ -1367,6 +1367,11 @@ IPCResult BrowserParent::RecvNewWindowGlobal(
   if (!browsingContext->Group()->IsKnownForChildID(OtherChildID())) {
     return IPC_FAIL(this, "Invalid BrowsingContextGroup for process");
   }
+  WindowGlobalParent* parentWgp = browsingContext->GetParentWindowContext();
+  if (browsingContext != mBrowsingContext &&
+      (!parentWgp || parentWgp->Manager() != this)) {
+    return IPC_FAIL(this, "BrowsingContext is not in BrowserParent subtree");
+  }
   if (!aInit.principal()) {
     return IPC_FAIL(this, "Cannot create without valid principal");
   }
@@ -1375,7 +1380,6 @@ IPCResult BrowserParent::RecvNewWindowGlobal(
   }
 
   nsCOMPtr<nsIURI> docURI = aInit.documentURI();
-  WindowGlobalParent* parentWgp = browsingContext->GetParentWindowContext();
 
   // Ensure we never load a document with a content principal in
   // the wrong type of webIsolated process
Loading diff…