Commit 319c61a3 authored by Mike Frysinger's avatar Mike Frysinger Committed by Commit Bot

zygote: stop using suid sandbox for oom adjust on CrOS

The patch to the kernel that required this usage has been dropped,
so we can drop this suid usage on CrOS now.

Bug: 576409, 850457
Change-Id: I8ea9321f4f233618a239db62a1ecf7a132bc54c8
Reviewed-on: https://chromium-review.googlesource.com/1112815Reviewed-by: default avatarJorge Lucangeli Obes <jorgelo@chromium.org>
Reviewed-by: default avatarGreg Kerr <kerrnel@chromium.org>
Reviewed-by: default avatarChris Palmer <palmer@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570843}
parent 9f818c41
......@@ -102,17 +102,6 @@ void ZygoteHostImpl::Init(const base::CommandLine& command_line) {
service_manager::switches::kDisableNamespaceSandbox) &&
sandbox::Credentials::CanCreateProcessInNewUserNS()) {
use_namespace_sandbox_ = true;
#if defined(OS_CHROMEOS)
// Chrome OS has a kernel patch that restricts oom_score_adj. See
// https://crbug.com/576409 for details.
if (!sandbox_binary_.empty()) {
use_suid_sandbox_for_adj_oom_score_ = true;
} else {
LOG(ERROR) << "SUID sandbox binary is missing. Won't be able to adjust "
"OOM scores.";
}
#endif
} else if (!command_line.HasSwitch(
service_manager::switches::kDisableSetuidSandbox) &&
!sandbox_binary_.empty()) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment