Correctly determine if the SUID sandbox will be used for adjusting the OOM score

content/browser/zygote_host/zygote_host_impl_linux.cc

 // Copyright (c) 2012 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.
 
 #include "content/browser/zygote_host/zygote_host_impl_linux.h"
 
 #include "base/allocator/allocator_extension.h"
 #include "base/command_line.h"
 #include "base/files/file_enumerator.h"
 #include "base/process/kill.h"
@@ skipping 32 matching lines @@
       *base::CommandLine::ForCurrentProcess();
   if (command_line.HasSwitch(switches::kNoSandbox) ||
       command_line.HasSwitch(switches::kDisableNamespaceSandbox) ||
       !sandbox::Credentials::CanCreateProcessInNewUserNS()) {
     should_use_namespace_sandbox_ = false;
   }
 
   const bool using_namespace_sandbox = ShouldUseNamespaceSandbox();
   // A non empty sandbox_cmd means we want a SUID sandbox.
   const bool using_suid_sandbox =
-      sandbox_binary_.empty() && !using_namespace_sandbox;
+      !sandbox_binary_.empty() && !using_namespace_sandbox;
 
   // Use the SUID sandbox for adjusting OOM scores when we are using the setuid
   // sandbox. This is needed beacuse the processes are non-dumpable, so
   // /proc/pid/oom_score_adj can only be written by root.
   use_suid_sandbox_for_adj_oom_score_ = using_suid_sandbox;
 
 #if defined(OS_CHROMEOS)
   // Chrome OS has a kernel patch that restricts oom_score_adj. See
   // crbug.com/576409 for details.
   if (!sandbox_binary_.empty()) {
@@ skipping 94 matching lines @@
     if (sandbox_helper_process.IsValid())
       base::EnsureProcessGetsReaped(sandbox_helper_process.Pid());
   } else if (!use_suid_sandbox_for_adj_oom_score_) {
     if (!base::AdjustOOMScore(pid, score))
       PLOG(ERROR) << "Failed to adjust OOM score of renderer with pid " << pid;
   }
 }
 #endif
 
 }  // namespace content