Electron: Sandboxed iframe can bypass the allow-popups restriction via the OpenURL navigation path
Description
Impact
A sandboxed iframe without the allow-popups keyword could still open a new window (or trigger setWindowOpenHandler) with no user interaction, because new-window navigations taking the OpenURL path did not apply the iframe sandbox popup restriction.
Apps that embed untrusted content in sandboxed iframes and rely on the absence of allow-popups to prevent window creation are affected. Apps that deny window creation in setWindowOpenHandler, or that do not embed untrusted content in sandboxed iframes, are not affected.
Workarounds
Return { action: 'deny' } from setWindowOpenHandler for any content you do not trust, rather than relying on the iframe sandbox alone.
Fixed
Versions * 42.0.1 * 41.10.3 * 39.8.10
For more information
If you have any questions or comments about this advisory, email Electron at [email protected]
Affected packages
Versions sourced from the GitHub Security Advisory.
| Package | Affected versions | Patched versions |
|---|---|---|
electronnpm | >= 42.0.0-alpha.1, < 42.0.1 | 42.0.1 |
electronnpm | >= 40.0.0-alpha.1, < 41.10.3 | 41.10.3 |
electronnpm | < 39.8.10 | 39.8.10 |
Affected products
1- Range: before 42.0.1, before 41.10.3, before 39.8.10
Patches
Vulnerability mechanics
References
11- github.com/advisories/GHSA-9f4c-93c8-jc8gghsaADVISORY
- github.com/electron/electron/commit/3ff23c52ab364a0afc6ab5bd7851291d3159de57ghsaWEB
- github.com/electron/electron/commit/57cbe329c4ae8aab5ac5ebdcb588adc9a11de0d3ghsaWEB
- github.com/electron/electron/commit/68cf8b7d9122260f6b534a69a82c701a56cf159fghsaWEB
- github.com/electron/electron/pull/51437ghsaWEB
- github.com/electron/electron/pull/51438ghsaWEB
- github.com/electron/electron/pull/51439ghsaWEB
- github.com/electron/electron/releases/tag/v39.8.10ghsaWEB
- github.com/electron/electron/releases/tag/v41.10.3ghsaWEB
- github.com/electron/electron/releases/tag/v42.0.1ghsaWEB
- github.com/electron/electron/security/advisories/GHSA-9f4c-93c8-jc8gghsaWEB
News mentions
0No linked articles in our index yet.