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::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; } diff --git a/src/processor/postfix_evaluator-inl.h b/src/processor/postfix_evaluator-inl.h index 0ebb57d6..06314125 100644 --- a/src/processor/postfix_evaluator-inl.h +++ b/src/processor/postfix_evaluator-inl.h @@ -162,8 +162,8 @@ bool PostfixEvaluator::EvaluateInternal( // = for assignment. ValueType value; if (!PopValue(&value)) { - BPLOG(ERROR) << "Could not PopValue to get value to assign: " << - expression; + BPLOG(INFO) << "Could not PopValue to get value to assign: " << + expression; return false; } @@ -212,7 +212,7 @@ bool PostfixEvaluator::Evaluate(const string &expression, // and successful. if (stack_.empty()) return true; - + BPLOG(ERROR) << "Incomplete execution: " << expression; return false; } @@ -231,7 +231,7 @@ bool PostfixEvaluator::EvaluateForValue(const string &expression, BPLOG(ERROR) << "Expression yielded bad number of results: " << "'" << expression << "'"; return false; - } + } return PopValue(result); } @@ -299,7 +299,7 @@ bool PostfixEvaluator::PopValue(ValueType *value) { if (iterator == dictionary_->end()) { // The identifier wasn't found in the dictionary. Don't imply any // default value, just fail. - BPLOG(ERROR) << "Identifier " << token << " not in dictionary"; + BPLOG(INFO) << "Identifier " << token << " not in dictionary"; return false; }