Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(139)

Unified Diff: chrome/common/sandbox_policy.cc

Issue 7084010: Move sandbox_policy to content. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/sandbox_policy.cc
===================================================================
--- chrome/common/sandbox_policy.cc (revision 87093)
+++ chrome/common/sandbox_policy.cc (working copy)
@@ -1,470 +0,0 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/common/sandbox_policy.h"
-
-#include <string>
-
-#include "base/command_line.h"
-#include "base/debug/debugger.h"
-#include "base/debug/trace_event.h"
-#include "base/file_util.h"
-#include "base/logging.h"
-#include "base/path_service.h"
-#include "base/process_util.h"
-#include "base/stringprintf.h"
-#include "base/string_util.h"
-#include "base/win/windows_version.h"
-#include "content/browser/content_browser_client.h"
-#include "content/common/content_switches.h"
-#include "content/common/child_process_info.h"
-#include "content/common/debug_flags.h"
-#include "sandbox/src/sandbox.h"
-
-static sandbox::BrokerServices* g_broker_services = NULL;
-
-namespace {
-
-// The DLLs listed here are known (or under strong suspicion) of causing crashes
-// when they are loaded in the renderer. Note: at runtime we generate short
-// versions of the dll name only if the dll has an extension.
-const wchar_t* const kTroublesomeDlls[] = {
- L"adialhk.dll", // Kaspersky Internet Security.
- L"acpiz.dll", // Unknown.
- L"avgrsstx.dll", // AVG 8.
- L"babylonchromepi.dll", // Babylon translator.
- L"btkeyind.dll", // Widcomm Bluetooth.
- L"cmcsyshk.dll", // CMC Internet Security.
- L"cooliris.dll", // CoolIris.
- L"dockshellhook.dll", // Stardock Objectdock.
- L"googledesktopnetwork3.dll", // Google Desktop Search v5.
- L"fwhook.dll", // PC Tools Firewall Plus.
- L"hookprocesscreation.dll", // Blumentals Program protector.
- L"hookterminateapis.dll", // Blumentals and Cyberprinter.
- L"hookprintapis.dll", // Cyberprinter.
- L"imon.dll", // NOD32 Antivirus.
- L"ioloHL.dll", // Iolo (System Mechanic).
- L"kloehk.dll", // Kaspersky Internet Security.
- L"lawenforcer.dll", // Spyware-Browser AntiSpyware (Spybro).
- L"libdivx.dll", // DivX.
- L"lvprcinj01.dll", // Logitech QuickCam.
- L"madchook.dll", // Madshi (generic hooking library).
- L"mdnsnsp.dll", // Bonjour.
- L"moonsysh.dll", // Moon Secure Antivirus.
- L"npdivx32.dll", // DivX.
- L"npggNT.des", // GameGuard 2008.
- L"npggNT.dll", // GameGuard (older).
- L"oawatch.dll", // Online Armor.
- L"pavhook.dll", // Panda Internet Security.
- L"pavshook.dll", // Panda Antivirus.
- L"pavshookwow.dll", // Panda Antivirus.
- L"pctavhook.dll", // PC Tools Antivirus.
- L"pctgmhk.dll", // PC Tools Spyware Doctor.
- L"prntrack.dll", // Pharos Systems.
- L"radhslib.dll", // Radiant Naomi Internet Filter.
- L"radprlib.dll", // Radiant Naomi Internet Filter.
- L"rapportnikko.dll", // Trustware Rapport.
- L"rlhook.dll", // Trustware Bufferzone.
- L"rooksdol.dll", // Trustware Rapport.
- L"rpchromebrowserrecordhelper.dll", // RealPlayer.
- L"rpmainbrowserrecordplugin.dll", // RealPlayer.
- L"r3hook.dll", // Kaspersky Internet Security.
- L"sahook.dll", // McAfee Site Advisor.
- L"sbrige.dll", // Unknown.
- L"sc2hook.dll", // Supercopier 2.
- L"sguard.dll", // Iolo (System Guard).
- L"smum32.dll", // Spyware Doctor version 6.
- L"smumhook.dll", // Spyware Doctor version 5.
- L"ssldivx.dll", // DivX.
- L"syncor11.dll", // SynthCore Midi interface.
- L"systools.dll", // Panda Antivirus.
- L"tfwah.dll", // Threatfire (PC tools).
- L"ycwebcamerasource.ax", // Cyberlink Camera helper.
- L"wblind.dll", // Stardock Object desktop.
- L"wbhelp.dll", // Stardock Object desktop.
- L"winstylerthemehelper.dll" // Tuneup utilities 2006.
-};
-
-// Adds the policy rules for the path and path\ with the semantic |access|.
-// If |children| is set to true, we need to add the wildcard rules to also
-// apply the rule to the subfiles and subfolders.
-bool AddDirectory(int path, const wchar_t* sub_dir, bool children,
- sandbox::TargetPolicy::Semantics access,
- sandbox::TargetPolicy* policy) {
- FilePath directory;
- if (!PathService::Get(path, &directory))
- return false;
-
- if (sub_dir) {
- directory = directory.Append(sub_dir);
- file_util::AbsolutePath(&directory);
- }
-
- sandbox::ResultCode result;
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_FILES, access,
- directory.value().c_str());
- if (result != sandbox::SBOX_ALL_OK)
- return false;
-
- std::wstring directory_str = directory.value() + L"\\";
- if (children)
- directory_str += L"*";
- // Otherwise, add the version of the path that ends with a separator.
-
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_FILES, access,
- directory_str.c_str());
- if (result != sandbox::SBOX_ALL_OK)
- return false;
-
- return true;
-}
-
-// Adds the policy rules for the path and path\* with the semantic |access|.
-// We need to add the wildcard rules to also apply the rule to the subkeys.
-bool AddKeyAndSubkeys(std::wstring key,
- sandbox::TargetPolicy::Semantics access,
- sandbox::TargetPolicy* policy) {
- sandbox::ResultCode result;
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_REGISTRY, access,
- key.c_str());
- if (result != sandbox::SBOX_ALL_OK)
- return false;
-
- key += L"\\*";
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_REGISTRY, access,
- key.c_str());
- if (result != sandbox::SBOX_ALL_OK)
- return false;
-
- return true;
-}
-
-// Compares the loaded |module| file name matches |module_name|.
-bool IsExpandedModuleName(HMODULE module, const wchar_t* module_name) {
- wchar_t path[MAX_PATH];
- DWORD sz = ::GetModuleFileNameW(module, path, arraysize(path));
- if ((sz == arraysize(path)) || (sz == 0)) {
- // XP does not set the last error properly, so we bail out anyway.
- return false;
- }
- if (!::GetLongPathName(path, path, arraysize(path)))
- return false;
- FilePath fname(path);
- return (fname.BaseName().value() == module_name);
-}
-
-// Adds a single dll by |module_name| into the |policy| blacklist.
-// To minimize the list we only add an unload policy only if the dll is
-// also loaded in this process. All the injected dlls of interest do this.
-void BlacklistAddOneDll(const wchar_t* module_name,
- sandbox::TargetPolicy* policy) {
- HMODULE module = ::GetModuleHandleW(module_name);
- if (!module) {
- // The module could have been loaded with a 8.3 short name. We use
- // the most common case: 'thelongname.dll' becomes 'thelon~1.dll'.
- std::wstring name(module_name);
- size_t period = name.rfind(L'.');
- DCHECK_NE(std::string::npos, period);
- DCHECK_LE(3U, (name.size() - period));
- if (period <= 8)
- return;
- std::wstring alt_name = name.substr(0, 6) + L"~1";
- alt_name += name.substr(period, name.size());
- module = ::GetModuleHandleW(alt_name.c_str());
- if (!module)
- return;
- // We found it, but because it only has 6 significant letters, we
- // want to make sure it is the right one.
- if (!IsExpandedModuleName(module, module_name))
- return;
- // Found a match. We add both forms to the policy.
- policy->AddDllToUnload(alt_name.c_str());
- }
- policy->AddDllToUnload(module_name);
- VLOG(1) << "dll to unload found: " << module_name;
- return;
-}
-
-// Adds policy rules for unloaded the known dlls that cause chrome to crash.
-// Eviction of injected DLLs is done by the sandbox so that the injected module
-// does not get a chance to execute any code.
-void AddDllEvictionPolicy(sandbox::TargetPolicy* policy) {
- for (int ix = 0; ix != arraysize(kTroublesomeDlls); ++ix)
- BlacklistAddOneDll(kTroublesomeDlls[ix], policy);
-}
-
-// Adds the generic policy rules to a sandbox TargetPolicy.
-bool AddGenericPolicy(sandbox::TargetPolicy* policy) {
- sandbox::ResultCode result;
-
- // Add the policy for the pipes
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_FILES,
- sandbox::TargetPolicy::FILES_ALLOW_ANY,
- L"\\??\\pipe\\chrome.*");
- if (result != sandbox::SBOX_ALL_OK)
- return false;
-
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_NAMED_PIPES,
- sandbox::TargetPolicy::NAMEDPIPES_ALLOW_ANY,
- L"\\\\.\\pipe\\chrome.nacl.*");
- if (result != sandbox::SBOX_ALL_OK)
- return false;
-
- // Add the policy for debug message only in debug
-#ifndef NDEBUG
- FilePath app_dir;
- if (!PathService::Get(base::DIR_MODULE, &app_dir))
- return false;
-
- wchar_t long_path_buf[MAX_PATH];
- DWORD long_path_return_value = GetLongPathName(app_dir.value().c_str(),
- long_path_buf,
- MAX_PATH);
- if (long_path_return_value == 0 || long_path_return_value >= MAX_PATH)
- return false;
-
- string16 debug_message(long_path_buf);
- file_util::AppendToPath(&debug_message, L"debug_message.exe");
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_PROCESS,
- sandbox::TargetPolicy::PROCESS_MIN_EXEC,
- debug_message.c_str());
- if (result != sandbox::SBOX_ALL_OK)
- return false;
-#endif // NDEBUG
-
- return true;
-}
-
-// For the GPU process we gotten as far as USER_LIMITED. The next level
-// which is USER_RESTRICTED breaks both the DirectX backend and the OpenGL
-// backend. Note that the GPU process is connected to the interactive
-// desktop.
-// TODO(cpu): Lock down the sandbox more if possible.
-// TODO(apatrick): Use D3D9Ex to render windowless.
-bool AddPolicyForGPU(CommandLine*, sandbox::TargetPolicy* policy) {
- policy->SetJobLevel(sandbox::JOB_UNPROTECTED, 0);
-
- if (base::win::GetVersion() > base::win::VERSION_XP) {
- policy->SetTokenLevel(sandbox::USER_RESTRICTED_SAME_ACCESS,
- sandbox::USER_LIMITED);
- policy->SetDelayedIntegrityLevel(sandbox::INTEGRITY_LEVEL_LOW);
- } else {
- policy->SetTokenLevel(sandbox::USER_UNPROTECTED,
- sandbox::USER_LIMITED);
- }
-
- AddDllEvictionPolicy(policy);
- return true;
-}
-
-void AddPolicyForRenderer(sandbox::TargetPolicy* policy) {
- policy->SetJobLevel(sandbox::JOB_LOCKDOWN, 0);
-
- sandbox::TokenLevel initial_token = sandbox::USER_UNPROTECTED;
- if (base::win::GetVersion() > base::win::VERSION_XP) {
- // On 2003/Vista the initial token has to be restricted if the main
- // token is restricted.
- initial_token = sandbox::USER_RESTRICTED_SAME_ACCESS;
- }
-
- policy->SetTokenLevel(initial_token, sandbox::USER_LOCKDOWN);
- policy->SetDelayedIntegrityLevel(sandbox::INTEGRITY_LEVEL_LOW);
-
- bool use_winsta = !CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kDisableAltWinstation);
-
- if (sandbox::SBOX_ALL_OK != policy->SetAlternateDesktop(use_winsta)) {
- DLOG(WARNING) << "Failed to apply desktop security to the renderer";
- }
-
- AddDllEvictionPolicy(policy);
-}
-
-// The Pepper process as locked-down as a renderer execpt that it can
-// create the server side of chrome pipes.
-bool AddPolicyForPepperPlugin(sandbox::TargetPolicy* policy) {
- sandbox::ResultCode result;
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_NAMED_PIPES,
- sandbox::TargetPolicy::NAMEDPIPES_ALLOW_ANY,
- L"\\\\.\\pipe\\chrome.*");
- if (result != sandbox::SBOX_ALL_OK) {
- NOTREACHED();
- return false;
- }
- AddPolicyForRenderer(policy);
- return true;
-}
-
-} // namespace
-
-namespace sandbox {
-
-void InitBrokerServices(sandbox::BrokerServices* broker_services) {
- // TODO(abarth): DCHECK(CalledOnValidThread());
- // See <http://b/1287166>.
- CHECK(broker_services);
- CHECK(!g_broker_services);
- broker_services->Init();
- g_broker_services = broker_services;
-}
-
-base::ProcessHandle StartProcessWithAccess(CommandLine* cmd_line,
- const FilePath& exposed_dir) {
- base::ProcessHandle process = 0;
- const CommandLine& browser_command_line = *CommandLine::ForCurrentProcess();
- ChildProcessInfo::ProcessType type;
- std::string type_str = cmd_line->GetSwitchValueASCII(switches::kProcessType);
- if (type_str == switches::kRendererProcess) {
- type = ChildProcessInfo::RENDER_PROCESS;
- } else if (type_str == switches::kExtensionProcess) {
- // Extensions are just renderers with another name.
- type = ChildProcessInfo::RENDER_PROCESS;
- } else if (type_str == switches::kPluginProcess) {
- type = ChildProcessInfo::PLUGIN_PROCESS;
- } else if (type_str == switches::kWorkerProcess) {
- type = ChildProcessInfo::WORKER_PROCESS;
- } else if (type_str == switches::kNaClLoaderProcess) {
- type = ChildProcessInfo::NACL_LOADER_PROCESS;
- } else if (type_str == switches::kUtilityProcess) {
- type = ChildProcessInfo::UTILITY_PROCESS;
- } else if (type_str == switches::kNaClBrokerProcess) {
- type = ChildProcessInfo::NACL_BROKER_PROCESS;
- } else if (type_str == switches::kGpuProcess) {
- type = ChildProcessInfo::GPU_PROCESS;
- } else if (type_str == switches::kPpapiPluginProcess) {
- type = ChildProcessInfo::PPAPI_PLUGIN_PROCESS;
- } else {
- NOTREACHED();
- return 0;
- }
-
- TRACE_EVENT_BEGIN_ETW("StartProcessWithAccess", 0, type_str);
-
- // To decide if the process is going to be sandboxed we have two cases.
- // First case: all process types except the nacl broker, and the plugin
- // process are sandboxed by default.
- bool in_sandbox =
- (type != ChildProcessInfo::NACL_BROKER_PROCESS) &&
- (type != ChildProcessInfo::PLUGIN_PROCESS);
-
- // If it is the GPU process then it can be disabled by a command line flag.
- if ((type == ChildProcessInfo::GPU_PROCESS) &&
- (browser_command_line.HasSwitch(switches::kDisableGpuSandbox))) {
- in_sandbox = false;
- VLOG(1) << "GPU sandbox is disabled";
- }
-
- if (browser_command_line.HasSwitch(switches::kNoSandbox)) {
- // The user has explicity opted-out from all sandboxing.
- in_sandbox = false;
- }
-
-#if !defined (GOOGLE_CHROME_BUILD)
- if (browser_command_line.HasSwitch(switches::kInProcessPlugins)) {
- // In process plugins won't work if the sandbox is enabled.
- in_sandbox = false;
- }
-#endif
- if (!browser_command_line.HasSwitch(switches::kDisable3DAPIs) &&
- !browser_command_line.HasSwitch(switches::kDisableExperimentalWebGL) &&
- browser_command_line.HasSwitch(switches::kInProcessWebGL)) {
- // In process WebGL won't work if the sandbox is enabled.
- in_sandbox = false;
- }
-
- // Propagate the Chrome Frame flag to sandboxed processes if present.
- if (browser_command_line.HasSwitch(switches::kChromeFrame)) {
- if (!cmd_line->HasSwitch(switches::kChromeFrame)) {
- cmd_line->AppendSwitch(switches::kChromeFrame);
- }
- }
-
- bool child_needs_help =
- DebugFlags::ProcessDebugFlags(cmd_line, type, in_sandbox);
-
- // Prefetch hints on windows:
- // Using a different prefetch profile per process type will allow Windows
- // to create separate pretetch settings for browser, renderer etc.
- cmd_line->AppendArg(base::StringPrintf("/prefetch:%d", type));
-
- sandbox::ResultCode result;
- PROCESS_INFORMATION target = {0};
- sandbox::TargetPolicy* policy = g_broker_services->CreatePolicy();
-
- if (type == ChildProcessInfo::PLUGIN_PROCESS &&
- !browser_command_line.HasSwitch(switches::kNoSandbox) &&
- content::GetContentClient()->browser()->SandboxPlugin(cmd_line, policy)) {
- in_sandbox = true;
- AddDllEvictionPolicy(policy);
- }
-
- if (!in_sandbox) {
- policy->Release();
- base::LaunchApp(*cmd_line, false, false, &process);
- return process;
- }
-
- if (type == ChildProcessInfo::GPU_PROCESS) {
- if (!AddPolicyForGPU(cmd_line, policy))
- return 0;
- } else if (type == ChildProcessInfo::PPAPI_PLUGIN_PROCESS) {
- if (!AddPolicyForPepperPlugin(policy))
- return 0;
- } else {
- AddPolicyForRenderer(policy);
-
- if (type_str != switches::kRendererProcess) {
- // Hack for Google Desktop crash. Trick GD into not injecting its DLL into
- // this subprocess. See
- // http://code.google.com/p/chromium/issues/detail?id=25580
- cmd_line->AppendSwitchASCII("ignored", " --type=renderer ");
- }
- }
-
- if (!exposed_dir.empty()) {
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_FILES,
- sandbox::TargetPolicy::FILES_ALLOW_ANY,
- exposed_dir.value().c_str());
- if (result != sandbox::SBOX_ALL_OK)
- return 0;
-
- FilePath exposed_files = exposed_dir.AppendASCII("*");
- result = policy->AddRule(sandbox::TargetPolicy::SUBSYS_FILES,
- sandbox::TargetPolicy::FILES_ALLOW_ANY,
- exposed_files.value().c_str());
- if (result != sandbox::SBOX_ALL_OK)
- return 0;
- }
-
- if (!AddGenericPolicy(policy)) {
- NOTREACHED();
- return 0;
- }
-
- TRACE_EVENT_BEGIN_ETW("StartProcessWithAccess::LAUNCHPROCESS", 0, 0);
-
- result = g_broker_services->SpawnTarget(
- cmd_line->GetProgram().value().c_str(),
- cmd_line->command_line_string().c_str(),
- policy, &target);
- policy->Release();
-
- TRACE_EVENT_END_ETW("StartProcessWithAccess::LAUNCHPROCESS", 0, 0);
-
- if (sandbox::SBOX_ALL_OK != result)
- return 0;
-
- ResumeThread(target.hThread);
- CloseHandle(target.hThread);
- process = target.hProcess;
-
- // Help the process a little. It can't start the debugger by itself if
- // the process is in a sandbox.
- if (child_needs_help)
- base::debug::SpawnDebuggerOnProcess(target.dwProcessId);
-
- return process;
-}
-
-} // namespace sandbox

Powered by Google App Engine
This is Rietveld 408576698