summaryrefslogtreecommitdiffstats
path: root/sandbox
diff options
context:
space:
mode:
authorjln@chromium.org <jln@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-06-07 04:04:06 +0000
committerjln@chromium.org <jln@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-06-07 04:04:06 +0000
commit86dab57a9579d3863341bbea412ee263b631f556 (patch)
treeb96dd4aba48e2c59653703bb84dc082a3f21e739 /sandbox
parentc1285ae4272755a44471080616917e31e317d929 (diff)
downloadchromium_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.cc4
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;
}