Bug 1308564 - Tie logging to MOZ_SANDBOX_VERBOSE instead of DEBUG. r=jld
authorGian-Carlo Pascutto <gcp@mozilla.com>
Fri, 07 Oct 2016 21:33:12 +0200
changeset 317358 a74c02610e4d62b8231cd1043c19b2ff3a72bc0a
parent 317357 7078e9bd3895e041165d402cce0a5509cd10185b
child 317359 f6b04b718e50e3cb23a2c2b4affea056aea1b9df
push id30802
push usercbook@mozilla.com
push dateTue, 11 Oct 2016 09:57:58 +0000
treeherdermozilla-central@45052ddb03a4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjld
bugs1308564
milestone52.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1308564 - Tie logging to MOZ_SANDBOX_VERBOSE instead of DEBUG. r=jld MozReview-Commit-ID: JHIzIvbjA0Q
security/sandbox/linux/broker/SandboxBroker.cpp
--- a/security/sandbox/linux/broker/SandboxBroker.cpp
+++ b/security/sandbox/linux/broker/SandboxBroker.cpp
@@ -715,17 +715,17 @@ SandboxBroker::AuditPermissive(int aOp, 
   SANDBOX_LOG_ERROR("SandboxBroker: would have denied op=%d rflags=%o perms=%d path=%s for pid=%d" \
                     " permissive=1 error=\"%s\"", aOp, aFlags, aPerms,
                     aPath, mChildPid, strerror(errno));
 }
 
 void
 SandboxBroker::AuditDenial(int aOp, int aFlags, int aPerms, const char* aPath)
 {
-#ifdef DEBUG
-  SANDBOX_LOG_ERROR("SandboxBroker: denied op=%d rflags=%o perms=%d path=%s for pid=%d" \
-                    " error=\"%s\"", aOp, aFlags, aPerms, aPath, mChildPid,
-                    strerror(errno));
-#endif
+  if (SandboxInfo::Get().Test(SandboxInfo::kVerbose)) {
+    SANDBOX_LOG_ERROR("SandboxBroker: denied op=%d rflags=%o perms=%d path=%s for pid=%d" \
+                      " error=\"%s\"", aOp, aFlags, aPerms, aPath, mChildPid,
+                      strerror(errno));
+  }
 }
 
 
 } // namespace mozilla