123456789101112131415161718192021222324252627 |
- From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
- From: Jeremy Apthorp <[email protected]>
- Date: Mon, 26 Aug 2019 12:02:51 -0700
- Subject: allow new privileges in unsandboxed child processes
- This allows unsandboxed child process to launch setuid processes on Linux.
- diff --git a/content/browser/child_process_launcher_helper_linux.cc b/content/browser/child_process_launcher_helper_linux.cc
- index add20a2f81d66e51e1be5a3f3b35ef6b340b4cc3..31a2a14a95540477297943df9b09b1e4659a884d 100644
- --- a/content/browser/child_process_launcher_helper_linux.cc
- +++ b/content/browser/child_process_launcher_helper_linux.cc
- @@ -64,6 +64,15 @@ bool ChildProcessLauncherHelper::BeforeLaunchOnLauncherThread(
- options->fds_to_remap.emplace_back(sandbox_fd, GetSandboxFD());
- }
-
- + // (For Electron), if we're launching without zygote, that means we're
- + // launching an unsandboxed process (since all sandboxed processes are
- + // forked from the zygote). Relax the allow_new_privs option to permit
- + // launching suid processes from unsandboxed child processes.
- + if (!base::CommandLine::ForCurrentProcess()->HasSwitch(switches::kNoZygote) &&
- + delegate_->GetZygote() == nullptr) {
- + options->allow_new_privs = true;
- + }
- +
- options->environment = delegate_->GetEnvironment();
- } else {
- DCHECK(GetZygoteForLaunch());
|