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

Unified Diff: services/service_manager/runner/host/linux_sandbox.cc

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 years 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: services/service_manager/runner/host/linux_sandbox.cc
diff --git a/services/service_manager/runner/host/linux_sandbox.cc b/services/service_manager/runner/host/linux_sandbox.cc
index 46fc62fa58246935a4cac5af85649a3024371a9b..25b6798c2ac7c2edcad60f2f8900d64b3c728f19 100644
--- a/services/service_manager/runner/host/linux_sandbox.cc
+++ b/services/service_manager/runner/host/linux_sandbox.cc
@@ -151,9 +151,9 @@ void LinuxSandbox::EngageSeccompSandbox() {
openat(proc_fd_.get(), ".", O_RDONLY | O_DIRECTORY | O_CLOEXEC)));
CHECK(proc_fd.is_valid());
sandbox.SetProcFd(std::move(proc_fd));
+ // Starting the process with a sandbox failed. Missing kernel support.
CHECK(
- sandbox.StartSandbox(sandbox::SandboxBPF::SeccompLevel::SINGLE_THREADED))
- << "Starting the process with a sandbox failed. Missing kernel support.";
+ sandbox.StartSandbox(sandbox::SandboxBPF::SeccompLevel::SINGLE_THREADED));
// The Broker is now bound to this process and should only be destroyed when
// the process exits or is killed.

Powered by Google App Engine
This is Rietveld 408576698