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

Unified Diff: base/debug/stack_trace_posix.cc

Issue 3945002: Move debug-related stuff from base to the base/debug directory and use the... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 years, 2 months 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
« no previous file with comments | « base/debug/stack_trace.cc ('k') | base/debug/stack_trace_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/debug/stack_trace_posix.cc
===================================================================
--- base/debug/stack_trace_posix.cc (revision 0)
+++ base/debug/stack_trace_posix.cc (working copy)
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "base/debug_util.h"
+#include "base/debug/stack_trace.h"
#include <errno.h>
#include <fcntl.h>
@@ -39,7 +39,11 @@
#include "base/third_party/symbolize/symbolize.h"
#endif
+namespace base {
+namespace debug {
+
namespace {
+
// The prefix used for mangled symbols, per the Itanium C++ ABI:
// http://www.codesourcery.com/cxx-abi/abi.html#mangling
const char kMangledSymbolPrefix[] = "_Z";
@@ -145,128 +149,6 @@
} // namespace
-// static
-bool DebugUtil::SpawnDebuggerOnProcess(unsigned /* process_id */) {
- NOTIMPLEMENTED();
- return false;
-}
-
-#if defined(OS_MACOSX)
-
-// Based on Apple's recommended method as described in
-// http://developer.apple.com/qa/qa2004/qa1361.html
-// static
-bool DebugUtil::BeingDebugged() {
- // If the process is sandboxed then we can't use the sysctl, so cache the
- // value.
- static bool is_set = false;
- static bool being_debugged = false;
-
- if (is_set) {
- return being_debugged;
- }
-
- // Initialize mib, which tells sysctl what info we want. In this case,
- // we're looking for information about a specific process ID.
- int mib[] = {
- CTL_KERN,
- KERN_PROC,
- KERN_PROC_PID,
- getpid()
- };
-
- // Caution: struct kinfo_proc is marked __APPLE_API_UNSTABLE. The source and
- // binary interfaces may change.
- struct kinfo_proc info;
- size_t info_size = sizeof(info);
-
- int sysctl_result = sysctl(mib, arraysize(mib), &info, &info_size, NULL, 0);
- DCHECK_EQ(sysctl_result, 0);
- if (sysctl_result != 0) {
- is_set = true;
- being_debugged = false;
- return being_debugged;
- }
-
- // This process is being debugged if the P_TRACED flag is set.
- is_set = true;
- being_debugged = (info.kp_proc.p_flag & P_TRACED) != 0;
- return being_debugged;
-}
-
-#elif defined(OS_LINUX)
-
-// We can look in /proc/self/status for TracerPid. We are likely used in crash
-// handling, so we are careful not to use the heap or have side effects.
-// Another option that is common is to try to ptrace yourself, but then we
-// can't detach without forking(), and that's not so great.
-// static
-bool DebugUtil::BeingDebugged() {
- int status_fd = open("/proc/self/status", O_RDONLY);
- if (status_fd == -1)
- return false;
-
- // We assume our line will be in the first 1024 characters and that we can
- // read this much all at once. In practice this will generally be true.
- // This simplifies and speeds up things considerably.
- char buf[1024];
-
- ssize_t num_read = HANDLE_EINTR(read(status_fd, buf, sizeof(buf)));
- if (HANDLE_EINTR(close(status_fd)) < 0)
- return false;
-
- if (num_read <= 0)
- return false;
-
- base::StringPiece status(buf, num_read);
- base::StringPiece tracer("TracerPid:\t");
-
- base::StringPiece::size_type pid_index = status.find(tracer);
- if (pid_index == base::StringPiece::npos)
- return false;
-
- // Our pid is 0 without a debugger, assume this for any pid starting with 0.
- pid_index += tracer.size();
- return pid_index < status.size() && status[pid_index] != '0';
-}
-
-#elif defined(OS_FREEBSD)
-
-bool DebugUtil::BeingDebugged() {
- // TODO(benl): can we determine this under FreeBSD?
- NOTIMPLEMENTED();
- return false;
-}
-
-#endif // defined(OS_FREEBSD)
-
-// We want to break into the debugger in Debug mode, and cause a crash dump in
-// Release mode. Breakpad behaves as follows:
-//
-// +-------+-----------------+-----------------+
-// | OS | Dump on SIGTRAP | Dump on SIGABRT |
-// +-------+-----------------+-----------------+
-// | Linux | N | Y |
-// | Mac | Y | N |
-// +-------+-----------------+-----------------+
-//
-// Thus we do the following:
-// Linux: Debug mode, send SIGTRAP; Release mode, send SIGABRT.
-// Mac: Always send SIGTRAP.
-
-#if defined(NDEBUG) && !defined(OS_MACOSX)
-#define DEBUG_BREAK() abort()
-#elif defined(ARCH_CPU_ARM_FAMILY)
-#define DEBUG_BREAK() asm("bkpt 0")
-#else
-#define DEBUG_BREAK() asm("int3")
-#endif
-
-// static
-void DebugUtil::BreakDebugger() {
- DEBUG_BREAK();
-}
-
StackTrace::StackTrace() {
#if defined(OS_MACOSX) && MAC_OS_X_VERSION_MIN_REQUIRED < MAC_OS_X_VERSION_10_5
if (backtrace == NULL) {
@@ -312,3 +194,6 @@
(*os) << "\t" << trace_strings[i] << "\n";
}
}
+
+} // namespace debug
+} // namespace base
« no previous file with comments | « base/debug/stack_trace.cc ('k') | base/debug/stack_trace_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698