Skip to content

Commit

Permalink
AIR-8 : do not cancel stackwalk when module not found.
Browse files Browse the repository at this point in the history
  • Loading branch information
bozimmerman committed Jun 10, 2023
1 parent 92797dd commit 65c899e
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/processor/stack_frame_symbolizer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,9 @@ StackFrameSymbolizer::SymbolizerResult StackFrameSymbolizer::FillSourceLineInfo(
// If module is known to have missing symbol file, return.
if (no_symbol_modules_.find(module->code_file()) !=
no_symbol_modules_.end()) {
return kError;
//BZ: Previously, this would generate an error, which would cancel
// the remainder of the stackwalk.
return kNoError;
}

// If module is already loaded, go ahead to fill source line info and return.
Expand Down Expand Up @@ -112,9 +114,11 @@ StackFrameSymbolizer::SymbolizerResult StackFrameSymbolizer::FillSourceLineInfo(
return resolver_->IsModuleCorrupt(frame->module) ?
kWarningCorruptSymbols : kNoError;
} else {
//BZ: Previously, this would generate an error, which would cancel
// the remainder of the stackwalk.
BPLOG(ERROR) << "Failed to load symbol file in resolver.";
no_symbol_modules_.insert(module->code_file());
return kError;
return kNoError;
}
}

Expand Down

0 comments on commit 65c899e

Please sign in to comment.