diff options
author | jln@chromium.org <jln@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-07 04:04:06 +0000 |
---|---|---|
committer | jln@chromium.org <jln@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-07 04:04:06 +0000 |
commit | 86dab57a9579d3863341bbea412ee263b631f556 (patch) | |
tree | b96dd4aba48e2c59653703bb84dc082a3f21e739 /sandbox | |
parent | c1285ae4272755a44471080616917e31e317d929 (diff) | |
download | chromium_src-86dab57a9579d3863341bbea412ee263b631f556.zip chromium_src-86dab57a9579d3863341bbea412ee263b631f556.tar.gz chromium_src-86dab57a9579d3863341bbea412ee263b631f556.tar.bz2 |
BPF: Fix status_ not being set properly
- A typo prevented status_ from being set properly in
supportsSeccompSandbox()
- Also fix a warning by explicitly discarding a return value
BUG=
TEST=
NOTRY=true
Review URL: https://chromiumcodereview.appspot.com/10532038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140951 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sandbox')
-rw-r--r-- | sandbox/linux/seccomp-bpf/sandbox_bpf.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sandbox/linux/seccomp-bpf/sandbox_bpf.cc b/sandbox/linux/seccomp-bpf/sandbox_bpf.cc index a60b208..e7b1da6 100644 --- a/sandbox/linux/seccomp-bpf/sandbox_bpf.cc +++ b/sandbox/linux/seccomp-bpf/sandbox_bpf.cc @@ -94,7 +94,7 @@ Sandbox::SandboxStatus Sandbox::supportsSeccompSandbox(int proc_fd) { // In other words, if a single-threaded process is currently in the // STATUS_UNAVAILABLE state, it is safe to assume that sandboxing is // actually available. - status_ == STATUS_AVAILABLE; + status_ = STATUS_AVAILABLE; return status_; } @@ -169,7 +169,7 @@ bool Sandbox::isSingleThreaded(int proc_fd) { sb.st_nlink != 3 || HANDLE_EINTR(close(task))) { if (task >= 0) { - HANDLE_EINTR(close(task)); + (void) HANDLE_EINTR(close(task)); } return false; } |