diff options
| author | jessicag.feedback@gmail.com <jessicag.feedback@gmail.com@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2010-09-09 21:15:32 +0000 |
|---|---|---|
| committer | jessicag.feedback@gmail.com <jessicag.feedback@gmail.com@4c0a9323-5329-0410-9bdc-e9ce6186880e> | 2010-09-09 21:15:32 +0000 |
| commit | bb87ebd809e4437c953e9008a73927cd7e1c05ef (patch) | |
| tree | 6f290a282e7bcdc5fc88651b0e736bf5291a3fd1 /src/processor/minidump.cc | |
| parent | Remove proto generated source files from svn. (diff) | |
| download | breakpad-bb87ebd809e4437c953e9008a73927cd7e1c05ef.tar.xz | |
Reducing severity of most commonly logged ERRORS.
git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@681 4c0a9323-5329-0410-9bdc-e9ce6186880e
Diffstat (limited to 'src/processor/minidump.cc')
| -rw-r--r-- | src/processor/minidump.cc | 17 |
1 files changed, 10 insertions, 7 deletions
diff --git a/src/processor/minidump.cc b/src/processor/minidump.cc index 2398e9b9..0414a822 100644 --- a/src/processor/minidump.cc +++ b/src/processor/minidump.cc @@ -676,7 +676,7 @@ bool MinidumpContext::Read(u_int32_t expected_size) { } default: { - // Unknown context type - Don't log as an error yet. Let the + // Unknown context type - Don't log as an error yet. Let the // caller work that out. BPLOG(INFO) << "MinidumpContext unknown context type " << HexString(cpu_type); @@ -1185,11 +1185,12 @@ bool MinidumpMemoryRegion::GetMemoryAtAddressInternal(u_int64_t address, return false; } + // Common failure case if (address < descriptor_->start_of_memory_range || sizeof(T) > numeric_limits<u_int64_t>::max() - address || address + sizeof(T) > descriptor_->start_of_memory_range + descriptor_->memory.data_size) { - BPLOG(ERROR) << "MinidumpMemoryRegion request out of range: " << + BPLOG(INFO) << "MinidumpMemoryRegion request out of range: " << HexString(address) << "+" << sizeof(T) << "/" << HexString(descriptor_->start_of_memory_range) << "+" << HexString(descriptor_->memory.data_size); @@ -1482,7 +1483,7 @@ bool MinidumpThreadList::Read(u_int32_t expected_size) { } } - + if (thread_count > max_threads_) { BPLOG(ERROR) << "MinidumpThreadList count " << thread_count << " exceeds maximum " << max_threads_; @@ -1839,8 +1840,9 @@ string MinidumpModule::debug_file() const { } } - BPLOG_IF(ERROR, file.empty()) << "MinidumpModule could not determine " - "debug_file for " << *name_; + // Relatively common case + BPLOG_IF(INFO, file.empty()) << "MinidumpModule could not determine " + "debug_file for " << *name_; return file; } @@ -1907,8 +1909,9 @@ string MinidumpModule::debug_identifier() const { // TODO(mmentovai): on the Mac, provide fallbacks as in code_identifier(). - BPLOG_IF(ERROR, identifier.empty()) << "MinidumpModule could not determine " - "debug_identifier for " << *name_; + // Relatively common case + BPLOG_IF(INFO, identifier.empty()) << "MinidumpModule could not determine " + "debug_identifier for " << *name_; return identifier; } |