diff options
Diffstat (limited to 'src/processor/stackwalker_ppc.cc')
-rw-r--r-- | src/processor/stackwalker_ppc.cc | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/processor/stackwalker_ppc.cc b/src/processor/stackwalker_ppc.cc index 044ae99b..02a49b21 100644 --- a/src/processor/stackwalker_ppc.cc +++ b/src/processor/stackwalker_ppc.cc @@ -38,6 +38,7 @@ #include "google_breakpad/processor/call_stack.h" #include "google_breakpad/processor/memory_region.h" #include "google_breakpad/processor/stack_frame_cpu.h" +#include "processor/logging.h" namespace google_breakpad { @@ -54,14 +55,19 @@ StackwalkerPPC::StackwalkerPPC(const SystemInfo *system_info, // This implementation only covers 32-bit ppc CPUs. The limits of the // supplied stack are invalid. Mark memory_ = NULL, which will cause // stackwalking to fail. + BPLOG(ERROR) << "Memory out of range for stackwalking: " << + HexString(memory_->GetBase()) << "+" << + HexString(memory_->GetSize()); memory_ = NULL; } } StackFrame* StackwalkerPPC::GetContextFrame() { - if (!context_ || !memory_) + if (!context_ || !memory_) { + BPLOG(ERROR) << "Can't get context frame without context or memory"; return NULL; + } StackFramePPC *frame = new StackFramePPC(); @@ -78,8 +84,10 @@ StackFrame* StackwalkerPPC::GetContextFrame() { StackFrame* StackwalkerPPC::GetCallerFrame( const CallStack *stack, const vector< linked_ptr<StackFrameInfo> > &stack_frame_info) { - if (!memory_ || !stack) + if (!memory_ || !stack) { + BPLOG(ERROR) << "Can't get caller frame without memory or stack"; return NULL; + } // The instruction pointers for previous frames are saved on the stack. // The typical ppc calling convention is for the called procedure to store |