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

Unified Diff: sandbox/linux/services/namespace_sandbox.h

Issue 881733002: Add namespace sandbox class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Write the uid and gid maps in the pre_exec_delegate. Created 5 years, 11 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: sandbox/linux/services/namespace_sandbox.h
diff --git a/sandbox/linux/services/namespace_sandbox.h b/sandbox/linux/services/namespace_sandbox.h
new file mode 100644
index 0000000000000000000000000000000000000000..a3d824adb34b52b15acd37818becf51fd9b64e4d
--- /dev/null
+++ b/sandbox/linux/services/namespace_sandbox.h
@@ -0,0 +1,76 @@
+// Copyright 2015 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.
+
+#ifndef SANDBOX_LINUX_SERVICES_NAMESPACE_SANDBOX_H_
+#define SANDBOX_LINUX_SERVICES_NAMESPACE_SANDBOX_H_
+
+#include <sys/types.h>
+
+#include "base/command_line.h"
+#include "base/files/scoped_file.h"
+#include "base/macros.h"
+#include "base/process/launch.h"
+#include "base/process/process_handle.h"
+#include "sandbox/sandbox_export.h"
+
+namespace sandbox {
+
+// Helper class for starting a process inside a new user, PID, and network
+// namespace. Before using a namespace sandbox, check for namespaces support
+// using Credentials::CanCreateProcessInNewUserNS.
+//
+// A typical use for "A" launching a sandboxed process "B" would be:
+// 1. A sets up a command line for process B.
+// 2. A sets up launch options for B using mutable_launch_options.
+// 3. A launches B with Launch.
+// 4. B should be prepared to assume the role of init(1). In particular, apart
+// from SIGKILL and SIGSTOP, B cannot receive any signal for which it does
+// not have an explicit signal handler registered.
+// If B dies, all the processes in the namespace will die.
+// B can fork() and the parent can assume the role of init(1), by using
+// CreateInitProcessReaper().
+// 5. B chroots using Credentials::MoveToNewUserNS() and
+// Credentials::DropFileSystemAccess().
+//
+// An instance of this class may only be used once.
+class SANDBOX_EXPORT NamespaceSandbox {
+ public:
+ NamespaceSandbox();
+ ~NamespaceSandbox();
+
+ // Launch a new process inside its own user/PID/network namespaces (depending
+ // on kernel support). Requires at a minimum that user namespaces are
+ // supported (use Credentials::CanCreateProcessInNewUserNS to check this).
+ base::Process Launch(const base::CommandLine& cmdline);
+
+ // Returns a pointer to the launch options for the sandboxed process. The
+ // pre_exec_delegate and clone_flags fields of LaunchOptions should not be
+ // modified. Any pointers added to launch_options must be valid at the time
+ // that Launch is called.
+ base::LaunchOptions* mutable_launch_options() { return &launch_options_; }
jln (very slow on Chromium) 2015/02/03 01:14:30 Get rid of that and take LaunchOptions as a Launch
rickyz (no longer on Chrome) 2015/02/03 01:27:22 Done.
+
+ // Returns whether the namespace sandbox created a new user, PID, and network
+ // namespace. In particular, InNewUserNamespace should return true iff the
+ // process was started via this class.
+ static bool InNewUserNamespace();
+ static bool InNewPidNamespace();
+ static bool InNewNetNamespace();
+
+ private:
+ // Writes an mappings for the process's uid/gid to /proc/|pid|/{uid,gid}_map,
+ // then unblocks the process by writing to |fd|.
+ void WriteUidGidMapAndUnblockProcess(pid_t pid, base::ScopedFD fd);
+
+ // Whether Launch has been called.
+ bool launch_called_;
+
+ // Launch options for the child process.
+ base::LaunchOptions launch_options_;
+
+ DISALLOW_COPY_AND_ASSIGN(NamespaceSandbox);
+};
+
+} // namespace sandbox
+
+#endif // SANDBOX_LINUX_SERVICES_NAMESPACE_SANDBOX_H_

Powered by Google App Engine
This is Rietveld 408576698