diff options
| author | jln@chromium.org <jln@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-09 02:08:11 +0000 |
|---|---|---|
| committer | jln@chromium.org <jln@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-08-09 02:08:11 +0000 |
| commit | 5c874dc8225fbf8c0b4b73606485040eb2562ed0 (patch) | |
| tree | 73855ec3ceea9c171c473a55161e575a6ef3df39 /components | |
| parent | e5f96d7fed889c097d1bf7105c25315f6d52735c (diff) | |
| download | chromium_src-5c874dc8225fbf8c0b4b73606485040eb2562ed0.zip chromium_src-5c874dc8225fbf8c0b4b73606485040eb2562ed0.tar.gz chromium_src-5c874dc8225fbf8c0b4b73606485040eb2562ed0.tar.bz2 | |
NaCl Linux sandbox: enable SIGSYS in seccomp-bpf
Enable the seccomp-bpf SIGSYS handler as a crashing signal handler.
This makes debugging unsupported system calls emitted within the sandbox
a lot easier to debug.
BUG=168812
R=mseaborn@chromium.org
Review URL: https://codereview.chromium.org/22464005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216539 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'components')
| -rw-r--r-- | components/nacl/loader/nacl_sandbox_linux.cc | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/components/nacl/loader/nacl_sandbox_linux.cc b/components/nacl/loader/nacl_sandbox_linux.cc index 3041db6..c93aba7 100644 --- a/components/nacl/loader/nacl_sandbox_linux.cc +++ b/components/nacl/loader/nacl_sandbox_linux.cc @@ -137,11 +137,6 @@ bool InitializeBpfSandbox() { content::InitializeSandbox(NaClBpfSandboxPolicy); if (sandbox_is_initialized) { RunSandboxSanityChecks(); - // TODO(jln): Find a way to fix this. - // The sandbox' SIGSYS handler trips NaCl, so we disable it. - // If SIGSYS is triggered it'll now execute the default action - // (CORE). This will make it hard to track down bugs and sandbox violations. - CHECK(signal(SIGSYS, SIG_DFL) != SIG_ERR); return true; } return false; |