summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--base/rand_util_c.h3
-rw-r--r--chrome/browser/nacl-loader.sb39
-rwxr-xr-xchrome/chrome_browser.gypi1
-rw-r--r--chrome/common/sandbox_init_wrapper_mac.cc19
-rw-r--r--chrome/common/sandbox_mac.h10
-rw-r--r--chrome/common/sandbox_mac.mm38
-rw-r--r--chrome/renderer/renderer.sb16
-rw-r--r--chrome/test/nacl/nacl_test.cc4
8 files changed, 121 insertions, 9 deletions
diff --git a/base/rand_util_c.h b/base/rand_util_c.h
index 5a0bf73..20e3082 100644
--- a/base/rand_util_c.h
+++ b/base/rand_util_c.h
@@ -9,6 +9,9 @@
extern "C" {
#endif
+// Note this *should* be in "namespace base" but the function is needed
+// from C so namespaces cannot be used.
+
// Returns an FD for /dev/urandom, possibly pre-opened before sandboxing
// was switched on. This is a C function so that Native Client can use it.
int GetUrandomFD(void);
diff --git a/chrome/browser/nacl-loader.sb b/chrome/browser/nacl-loader.sb
new file mode 100644
index 0000000..61716a6
--- /dev/null
+++ b/chrome/browser/nacl-loader.sb
@@ -0,0 +1,39 @@
+;;
+;; Copyright (c) 2009 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.
+;;
+; This is the Sandbox configuration file used for safeguarding the user's
+; untrusted code within Native Client.
+;
+(version 1)
+(deny default)
+; Support for programmatically enabling verbose debugging.
+;ENABLE_LOGGING (debug deny)
+
+; Allow sending signals to self - http://crbug.com/20370
+(allow signal (target self))
+
+; Needed for full-page-zoomed controls - http://crbug.com/11325
+(allow sysctl-read)
+
+; Each line is marked with the System version that needs it.
+; This profile is tested with the following system versions:
+; 10.5.6, 10.6
+
+; Allow following symlinks
+(allow file-read-metadata) ; 10.5.6
+
+; Needed for IPC on 10.6
+;10.6_ONLY (allow ipc-posix-shm)
+
+; Needed for the Native Client plugin and loader.
+; TODO(msneck): Refactor Native Client to use something other than Unix
+; sockets.
+; See http://code.google.com/p/nativeclient/issues/detail?id=344
+;BEFORE_10.6 (allow network-inbound (from unix-socket))
+;BEFORE_10.6 (allow network-outbound (to unix-socket))
+;10.6_ONLY (allow network-inbound (regex #"^(/private)?/tmp/nacl-"))
+;10.6_ONLY (allow network-outbound (regex #"^(/private)?/tmp/nacl-"))
+;10.6_ONLY (allow network-bind (local ip4))
+;10.6_ONLY (allow file-write* (regex #"^(/private)?/tmp/nacl-"))
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi
index b748432..fae8a3a 100755
--- a/chrome/chrome_browser.gypi
+++ b/chrome/chrome_browser.gypi
@@ -2582,6 +2582,7 @@
'$(SDKROOT)/System/Library/Frameworks/OpenGL.framework',
],
'mac_bundle_resources': [
+ 'browser/nacl-loader.sb',
'browser/utility.sb',
'browser/worker.sb',
],
diff --git a/chrome/common/sandbox_init_wrapper_mac.cc b/chrome/common/sandbox_init_wrapper_mac.cc
index 898faed..3ed61ed 100644
--- a/chrome/common/sandbox_init_wrapper_mac.cc
+++ b/chrome/common/sandbox_init_wrapper_mac.cc
@@ -20,8 +20,17 @@ bool SandboxInitWrapper::InitializeSandbox(const CommandLine& command_line,
// Browser process isn't sandboxed.
return true;
} else if (process_type == switches::kRendererProcess) {
- // Renderer process sandbox.
- sandbox_process_type = sandbox::SANDBOX_TYPE_RENDERER;
+ // Renderer process sandbox. If --internal_nacl is present then use the
+ // version of the renderer sandbox which allows Native Client to use Unix
+ // sockets.
+ // TODO(msneck): Remove the use of Unix sockets from Native Client and
+ // then get rid of the SANDBOX_TYPE_NACL_PLUGIN enum.
+ // See http://code.google.com/p/nativeclient/issues/detail?id=344
+ if (command_line.HasSwitch(switches::kInternalNaCl)) {
+ sandbox_process_type = sandbox::SANDBOX_TYPE_NACL_PLUGIN;
+ } else {
+ sandbox_process_type = sandbox::SANDBOX_TYPE_RENDERER;
+ }
} else if (process_type == switches::kExtensionProcess) {
// Extension processes are just renderers [they use RenderMain()] with a
// different set of command line flags.
@@ -39,8 +48,10 @@ bool SandboxInitWrapper::InitializeSandbox(const CommandLine& command_line,
} else if (process_type == switches::kWorkerProcess) {
// Worker process sandbox.
sandbox_process_type = sandbox::SANDBOX_TYPE_WORKER;
- } else if ((process_type == switches::kNaClLoaderProcess) ||
- (process_type == switches::kPluginProcess) ||
+ } else if (process_type == switches::kNaClLoaderProcess) {
+ // Native Client sel_ldr (user untrusted code) sandbox.
+ sandbox_process_type = sandbox::SANDBOX_TYPE_NACL_LOADER;
+ } else if ((process_type == switches::kPluginProcess) ||
(process_type == switches::kProfileImportProcess) ||
(process_type == switches::kGpuProcess)) {
return true;
diff --git a/chrome/common/sandbox_mac.h b/chrome/common/sandbox_mac.h
index a8a55b0..c8ef4c3 100644
--- a/chrome/common/sandbox_mac.h
+++ b/chrome/common/sandbox_mac.h
@@ -12,12 +12,20 @@ namespace sandbox {
enum SandboxProcessType {
SANDBOX_TYPE_RENDERER,
- // Worker process has *everything* not needed for Cocoa locked down.
+ // The worker processes uses the most restrictive sandbox which has almost
+ // *everything* locked down. Only a couple of /System/Library/ paths and
+ // some other very basic operations (e.g., reading metadata to allow
+ // following symlinks) are permitted.
SANDBOX_TYPE_WORKER,
// Utility process is as restrictive as the worker process except full access
// is allowed to one configurable directory.
SANDBOX_TYPE_UTILITY,
+
+ // Native Client sandboxes. The plugin contains trusted code and the
+ // loader contains the user's untrusted code.
+ SANDBOX_TYPE_NACL_PLUGIN,
+ SANDBOX_TYPE_NACL_LOADER,
};
// Warm up System APIs that empirically need to be accessed before the Sandbox
diff --git a/chrome/common/sandbox_mac.mm b/chrome/common/sandbox_mac.mm
index ca8ac6c..1299481 100644
--- a/chrome/common/sandbox_mac.mm
+++ b/chrome/common/sandbox_mac.mm
@@ -15,6 +15,7 @@ extern "C" {
#include "base/command_line.h"
#include "base/file_util.h"
#include "base/mac_util.h"
+#include "base/rand_util_c.h"
#include "base/scoped_cftyperef.h"
#include "base/scoped_nsautorelease_pool.h"
#include "base/string16.h"
@@ -234,6 +235,10 @@ void SandboxWarmup() {
NULL));
CGImageSourceGetStatus(img);
}
+
+ { // Native Client access to /dev/random.
+ GetUrandomFD();
+ }
}
// Turns on the OS X sandbox for this process.
@@ -250,6 +255,7 @@ bool EnableSandbox(SandboxProcessType sandbox_type,
// TODO(jeremy): Look at using include syntax to unify common parts of sandbox
// definition files.
NSString* sandbox_config_filename = nil;
+ bool allow_nacl_lines = false;
switch (sandbox_type) {
case SANDBOX_TYPE_RENDERER:
sandbox_config_filename = @"renderer";
@@ -260,6 +266,26 @@ bool EnableSandbox(SandboxProcessType sandbox_type,
case SANDBOX_TYPE_UTILITY:
sandbox_config_filename = @"utility";
break;
+ case SANDBOX_TYPE_NACL_PLUGIN:
+ // The Native Client plugin is a standard renderer sandbox with some
+ // additional lines to support use of Unix sockets.
+ // TODO(msneck): Remove the use of Unix sockets from Native Client and
+ // then remove the associated rules from chrome/renderer/renderer.sb.
+ // See http://code.google.com/p/nativeclient/issues/detail?id=344
+ sandbox_config_filename = @"renderer";
+ allow_nacl_lines = true;
+ break;
+ case SANDBOX_TYPE_NACL_LOADER:
+ // The Native Client loader is used for safeguarding the user's
+ // untrusted code within Native Client.
+ // TODO(msneck): Remove the use of Unix sockets from Native Client and
+ // then decide on an appropriate sandbox type for the untrusted code.
+ // This might simply mean removing the Unix socket rules from
+ // chrome/browser/nacl-loader.sb or it might mean sharing the
+ // sandbox configuration with SANDBOX_TYPE_WORKER.
+ // See http://code.google.com/p/nativeclient/issues/detail?id=344
+ sandbox_config_filename = @"nacl-loader";
+ break;
default:
NOTREACHED();
return false;
@@ -288,6 +314,13 @@ bool EnableSandbox(SandboxProcessType sandbox_type,
withString:@""];
}
+ // Enable Native Client lines if they are allowed.
+ if (allow_nacl_lines) {
+ sandbox_data = [sandbox_data
+ stringByReplacingOccurrencesOfString:@";NACL"
+ withString:@""];
+ }
+
if (!allowed_dir.empty()) {
// The sandbox only understands "real" paths. This resolving step is
// needed so the caller doesn't need to worry about things like /var
@@ -341,6 +374,11 @@ bool EnableSandbox(SandboxProcessType sandbox_type,
sandbox_data = [sandbox_data
stringByReplacingOccurrencesOfString:@"USER_HOMEDIR"
withString:home_dir_escaped_ns];
+ } else if (major_version == 10 && minor_version < 6) {
+ // Sandbox rules only for versions before 10.6.
+ sandbox_data = [sandbox_data
+ stringByReplacingOccurrencesOfString:@";BEFORE_10.6"
+ withString:@""];
}
char* error_buff = NULL;
diff --git a/chrome/renderer/renderer.sb b/chrome/renderer/renderer.sb
index 27b2a90c..bbdf1c2 100644
--- a/chrome/renderer/renderer.sb
+++ b/chrome/renderer/renderer.sb
@@ -32,8 +32,22 @@
(allow mach-lookup (global-name "com.apple.FontObjectsServer")) ; 10.5.6
;10.6_ONLY (allow mach-lookup (global-name "com.apple.FontServer")) ; 10.6
-; USER_HOMEDIR is substitued at runtime - http://crbug.com/11269
+; USER_HOMEDIR is substitued at runtime - http://crbug.com/11269
;10.6_ONLY (allow file-read-data (subpath "USER_HOMEDIR/Library/Fonts")) ; 10.6
; Needed for IPC on 10.6
;10.6_ONLY (allow ipc-posix-shm)
+
+; Needed for the Native Client plugin and loader. These lines are enabled
+; if and only if --internal-nacl (or --enable-nacl) are used (and they
+; are off by default).
+; TODO(msneck): Refactor Native Client to use something other than Unix
+; sockets. Then change or remove the code in chrome/common/sandbox_mac.mm
+; which deals with the ";NACL" prefix.
+; See http://code.google.com/p/nativeclient/issues/detail?id=344
+;NACL;BEFORE_10.6 (allow network-inbound (from unix-socket))
+;NACL;BEFORE_10.6 (allow network-outbound (to unix-socket))
+;NACL;10.6_ONLY (allow network-inbound (regex #"^(/private)?/tmp/nacl-"))
+;NACL;10.6_ONLY (allow network-outbound (regex #"^(/private)?/tmp/nacl-"))
+;NACL;10.6_ONLY (allow network-bind (local ip4))
+;NACL;10.6_ONLY (allow file-write* (regex #"^(/private)?/tmp/nacl-"))
diff --git a/chrome/test/nacl/nacl_test.cc b/chrome/test/nacl/nacl_test.cc
index 7d2b6de..1ae70f9 100644
--- a/chrome/test/nacl/nacl_test.cc
+++ b/chrome/test/nacl/nacl_test.cc
@@ -62,9 +62,7 @@ NaClTest::NaClTest()
// Currently we disable some of the sandboxes. See:
// Make NaCl work in Chromium's Linux seccomp sandbox and the Mac sandbox
// http://code.google.com/p/nativeclient/issues/detail?id=344
-#if defined(OS_MACOSX)
- launch_arguments_.AppendSwitch(switches::kNoSandbox);
-#elif defined(OS_LINUX) && defined(USE_SECCOMP_SANDBOX)
+#if defined(OS_LINUX) && defined(USE_SECCOMP_SANDBOX)
launch_arguments_.AppendSwitch(switches::kDisableSeccompSandbox);
#endif
}
generated by cgit v1.1 at 2025-01-16 07:08:56 +0000