diff options
author | jschuh@chromium.org <jschuh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-09-13 16:01:21 +0000 |
---|---|---|
committer | jschuh@chromium.org <jschuh@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-09-13 16:01:21 +0000 |
commit | 7a3a69a2c0f5acb6c16f104399aecc0e85474f28 (patch) | |
tree | 3acd08f19dafe8150d045e877182cd474356edeb /sandbox/win/src/broker_services.cc | |
parent | 58c293847945c02725e211b1625722127ace14a5 (diff) | |
download | chromium_src-7a3a69a2c0f5acb6c16f104399aecc0e85474f28.zip chromium_src-7a3a69a2c0f5acb6c16f104399aecc0e85474f28.tar.gz chromium_src-7a3a69a2c0f5acb6c16f104399aecc0e85474f28.tar.bz2 |
Revert 156550 - Add sandbox support for Windows process mitigations
BUG=147752
Review URL: https://codereview.chromium.org/10690058
TBR=jschuh@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10907217
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156556 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sandbox/win/src/broker_services.cc')
-rw-r--r-- | sandbox/win/src/broker_services.cc | 37 |
1 files changed, 6 insertions, 31 deletions
diff --git a/sandbox/win/src/broker_services.cc b/sandbox/win/src/broker_services.cc index 0425845..497f2f8 100644 --- a/sandbox/win/src/broker_services.cc +++ b/sandbox/win/src/broker_services.cc @@ -12,7 +12,6 @@ #include "base/win/startup_information.h" #include "base/win/windows_version.h" #include "sandbox/win/src/app_container.h" -#include "sandbox/win/src/process_mitigations.h" #include "sandbox/win/src/sandbox_policy_base.h" #include "sandbox/win/src/sandbox.h" #include "sandbox/win/src/target_process.h" @@ -321,36 +320,12 @@ ResultCode BrokerServicesBase::SpawnTarget(const wchar_t* exe_path, const_cast<wchar_t*>(desktop.c_str()); } - if (base::win::GetVersion() >= base::win::VERSION_VISTA) { - int attribute_count = 0; - const AppContainerAttributes* app_container = - policy_base->GetAppContainer(); - if (app_container) - ++attribute_count; - - DWORD64 mitigations; - size_t mitigations_size; - ConvertProcessMitigationsToPolicy(policy->GetProcessMitigations(), - &mitigations, &mitigations_size); - if (mitigations) - ++attribute_count; - - if (!startup_info.InitializeProcThreadAttributeList(attribute_count)) - return SBOX_ERROR_PROC_THREAD_ATTRIBUTES; - - if (app_container) { - result = app_container->ShareForStartup(&startup_info); - if (SBOX_ALL_OK != result) - return result; - } - - if (mitigations) { - if (!startup_info.UpdateProcThreadAttribute( - PROC_THREAD_ATTRIBUTE_MITIGATION_POLICY, &mitigations, - mitigations_size)) { - return SBOX_ERROR_PROC_THREAD_ATTRIBUTES; - } - } + const AppContainerAttributes* app_container = policy_base->GetAppContainer(); + if (app_container) { + startup_info.InitializeProcThreadAttributeList(1); + result = app_container->ShareForStartup(&startup_info); + if (SBOX_ALL_OK != result) + return result; } // Construct the thread pool here in case it is expensive. |