aboutsummaryrefslogtreecommitdiff
path: root/src/processor/stackwalker_ppc.cc
diff options
context:
space:
mode:
Diffstat (limited to 'src/processor/stackwalker_ppc.cc')
-rw-r--r--src/processor/stackwalker_ppc.cc12
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