Skip to content
This repository has been archived by the owner on Aug 4, 2022. It is now read-only.

Commit

Permalink
Bug 1553249: Only set the Win32k disable policy for the RDD process w…
Browse files Browse the repository at this point in the history
…hen running on Win8+; r=bobowen

Differential Revision: https://phabricator.services.mozilla.com/D36239
  • Loading branch information
dblohm7 committed Jun 28, 2019
1 parent 3c87bf4 commit de5dc64
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion security/sandbox/win/src/sandboxbroker/sandboxBroker.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -792,7 +792,9 @@ bool SandboxBroker::SetSecurityLevelForRDDProcess() {
sandbox::MITIGATION_DEP_NO_ATL_THUNK | sandbox::MITIGATION_DEP |
sandbox::MITIGATION_IMAGE_LOAD_PREFER_SYS32;

if (sRddWin32kDisable) {
// On Windows 7, where Win32k lockdown is not supported, the Chromium
// sandbox does something weird that breaks COM instantiation.
if (sRddWin32kDisable && IsWin8OrLater()) {
mitigations |= sandbox::MITIGATION_WIN32K_DISABLE;
result =
mPolicy->AddRule(sandbox::TargetPolicy::SUBSYS_WIN32K_LOCKDOWN,
Expand Down

0 comments on commit de5dc64

Please sign in to comment.