aboutsummaryrefslogtreecommitdiff
path: root/src/tools/mac
diff options
context:
space:
mode:
authorwaylonis <waylonis@4c0a9323-5329-0410-9bdc-e9ce6186880e>2007-01-24 00:28:24 +0000
committerwaylonis <waylonis@4c0a9323-5329-0410-9bdc-e9ce6186880e>2007-01-24 00:28:24 +0000
commit4b987df62b8cf8eb529bff6a18d71a3832bf55c2 (patch)
treebac098f2214cbbd076cc59a30b01efbbe8cf6b1f /src/tools/mac
parentChanges to support patch #108: (diff)
downloadbreakpad-4b987df62b8cf8eb529bff6a18d71a3832bf55c2.tar.xz
Update crash_report tool and internal classes to reflect change to ProcessState (Issue #116)
git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@110 4c0a9323-5329-0410-9bdc-e9ce6186880e
Diffstat (limited to 'src/tools/mac')
-rw-r--r--src/tools/mac/crash_report/crash_report.mm41
-rw-r--r--src/tools/mac/crash_report/crash_report.xcodeproj/project.pbxproj6
-rw-r--r--src/tools/mac/crash_report/on_demand_symbol_supplier.h12
-rw-r--r--src/tools/mac/crash_report/on_demand_symbol_supplier.mm34
4 files changed, 37 insertions, 56 deletions
diff --git a/src/tools/mac/crash_report/crash_report.mm b/src/tools/mac/crash_report/crash_report.mm
index 1a42fc03..355c9dd3 100644
--- a/src/tools/mac/crash_report/crash_report.mm
+++ b/src/tools/mac/crash_report/crash_report.mm
@@ -47,6 +47,7 @@
#include "google_airbag/processor/minidump_processor.h"
#include "google_airbag/processor/process_state.h"
#include "google_airbag/processor/stack_frame_cpu.h"
+#include "google_airbag/processor/system_info.h"
#include "processor/pathname_stripper.h"
#include "processor/scoped_ptr.h"
#include "processor/simple_symbol_supplier.h"
@@ -67,6 +68,7 @@ using google_airbag::scoped_ptr;
using google_airbag::StackFrame;
using google_airbag::StackFramePPC;
using google_airbag::StackFrameX86;
+using google_airbag::SystemInfo;
typedef struct {
NSString *minidumpPath;
@@ -99,7 +101,7 @@ static void PrintStack(const CallStack *stack, const string &cpu) {
printf("%-*s", maxStr, buffer);
u_int64_t instruction = frame->instruction;
- // PPC only: Adjust the instruction to match that of Crash reporter. The
+ // PPC only: Adjust the instruction to match that of Crash reporter. The
// instruction listed is actually the return address. See the detailed
// comments in stackwalker_ppc.cc for more information.
if (cpu == "ppc" && frame_index)
@@ -186,25 +188,12 @@ static void PrintRegisters(const CallStack *stack, const string &cpu) {
static void Start(Options *options) {
string minidump_file([options->minidumpPath fileSystemRepresentation]);
- // Guess that the symbols are for our default architecture
- const NXArchInfo *localArchInfo = NXGetLocalArchInfo();
- string arch = "ppc";
-
- if (!localArchInfo)
- return;
-
- if (localArchInfo->cputype & CPU_ARCH_ABI64)
- arch = ((localArchInfo->cputype & ~CPU_ARCH_ABI64) == CPU_TYPE_X86) ?
- "x86_64" : "ppc64";
- else
- arch = (localArchInfo->cputype == CPU_TYPE_X86) ? "x86" : "ppc";
-
BasicSourceLineResolver resolver;
string search_dir = options->searchDir ?
[options->searchDir fileSystemRepresentation] : "";
scoped_ptr<OnDemandSymbolSupplier> symbol_supplier(
- new OnDemandSymbolSupplier(arch, search_dir));
- scoped_ptr<MinidumpProcessor>
+ new OnDemandSymbolSupplier(search_dir));
+ scoped_ptr<MinidumpProcessor>
minidump_processor(new MinidumpProcessor(symbol_supplier.get(), &resolver));
ProcessState process_state;
if (minidump_processor->Process(minidump_file, &process_state) !=
@@ -213,19 +202,8 @@ static void Start(Options *options) {
return;
}
- string cpu = process_state.cpu();
- // If the minidump is different than the default architecture
- if (cpu != arch) {
- symbol_supplier.reset(new OnDemandSymbolSupplier(cpu, search_dir));
- minidump_processor.reset(new MinidumpProcessor(symbol_supplier.get(),
- &resolver));
-
- if (minidump_processor->Process(minidump_file, &process_state) !=
- MinidumpProcessor::PROCESS_OK) {
- fprintf(stderr, "MinidumpProcessor::Process failed\n");
- return;
- }
- }
+ const SystemInfo *system_info = process_state.system_info();
+ string cpu = system_info->cpu;
// Convert the time to a string
u_int32_t time_date_stamp = process_state.time_date_stamp();
@@ -235,9 +213,8 @@ static void Start(Options *options) {
strftime(timestr, 20, "%Y-%m-%d %H:%M:%S", &timestruct);
printf("Date: %s GMT\n", timestr);
- string cpu_info = process_state.cpu_info();
- printf("Operating system: %s (%s)\n", process_state.os().c_str(),
- process_state.os_version().c_str());
+ printf("Operating system: %s (%s)\n", system_info->os.c_str(),
+ system_info->os_version.c_str());
printf("Architecture: %s\n", cpu.c_str());
if (process_state.crashed()) {
diff --git a/src/tools/mac/crash_report/crash_report.xcodeproj/project.pbxproj b/src/tools/mac/crash_report/crash_report.xcodeproj/project.pbxproj
index eb43b3a8..24459d13 100644
--- a/src/tools/mac/crash_report/crash_report.xcodeproj/project.pbxproj
+++ b/src/tools/mac/crash_report/crash_report.xcodeproj/project.pbxproj
@@ -17,6 +17,7 @@
9B3904960B2E52D90059FABE /* basic_source_line_resolver.h in CopyFiles */ = {isa = PBXBuildFile; fileRef = 9B3904940B2E52D90059FABE /* basic_source_line_resolver.h */; };
9B3904970B2E52D90059FABE /* source_line_resolver_interface.h in CopyFiles */ = {isa = PBXBuildFile; fileRef = 9B3904950B2E52D90059FABE /* source_line_resolver_interface.h */; };
9B3904990B2E52FD0059FABE /* basic_source_line_resolver.cc in Sources */ = {isa = PBXBuildFile; fileRef = 9B3904980B2E52FD0059FABE /* basic_source_line_resolver.cc */; };
+ 9B44619E0B66C66B00BBB817 /* system_info.h in CopyFiles */ = {isa = PBXBuildFile; fileRef = 9B44619D0B66C66B00BBB817 /* system_info.h */; };
9BDF172C0B1B8B2400F8391B /* call_stack.cc in Sources */ = {isa = PBXBuildFile; fileRef = 9BDF172A0B1B8B2400F8391B /* call_stack.cc */; };
9BDF172D0B1B8B2400F8391B /* minidump_processor.cc in Sources */ = {isa = PBXBuildFile; fileRef = 9BDF172B0B1B8B2400F8391B /* minidump_processor.cc */; };
9BDF17410B1B8B9A00F8391B /* minidump.cc in Sources */ = {isa = PBXBuildFile; fileRef = 9BDF173F0B1B8B9A00F8391B /* minidump.cc */; };
@@ -63,6 +64,7 @@
9BE650B30B52FE3000611104 /* file_id.h in CopyFiles */,
9BE650B50B52FE3000611104 /* macho_id.h in CopyFiles */,
9BE650B70B52FE3000611104 /* macho_walker.h in CopyFiles */,
+ 9B44619E0B66C66B00BBB817 /* system_info.h in CopyFiles */,
);
runOnlyForDeploymentPostprocessing = 1;
};
@@ -80,6 +82,7 @@
9B3904940B2E52D90059FABE /* basic_source_line_resolver.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = basic_source_line_resolver.h; sourceTree = "<group>"; };
9B3904950B2E52D90059FABE /* source_line_resolver_interface.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = source_line_resolver_interface.h; sourceTree = "<group>"; };
9B3904980B2E52FD0059FABE /* basic_source_line_resolver.cc */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.cpp.cpp; name = basic_source_line_resolver.cc; path = ../../../processor/basic_source_line_resolver.cc; sourceTree = SOURCE_ROOT; };
+ 9B44619D0B66C66B00BBB817 /* system_info.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = system_info.h; sourceTree = "<group>"; };
9BDF16F90B1B8ACD00F8391B /* airbag_types.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = airbag_types.h; sourceTree = "<group>"; };
9BDF16FA0B1B8ACD00F8391B /* minidump_format.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = minidump_format.h; sourceTree = "<group>"; };
9BDF16FC0B1B8ACD00F8391B /* call_stack.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = call_stack.h; sourceTree = "<group>"; };
@@ -191,7 +194,6 @@
isa = PBXGroup;
children = (
9B3904940B2E52D90059FABE /* basic_source_line_resolver.h */,
- 9B3904950B2E52D90059FABE /* source_line_resolver_interface.h */,
9BDF16FC0B1B8ACD00F8391B /* call_stack.h */,
9B35FEE20B2675F9008DE8C7 /* code_module.h */,
9B35FEE30B2675F9008DE8C7 /* code_modules.h */,
@@ -199,10 +201,12 @@
9BDF16FE0B1B8ACD00F8391B /* minidump.h */,
9BDF16FF0B1B8ACD00F8391B /* minidump_processor.h */,
9BDF17000B1B8ACD00F8391B /* process_state.h */,
+ 9B3904950B2E52D90059FABE /* source_line_resolver_interface.h */,
9BDF17010B1B8ACD00F8391B /* stack_frame.h */,
9BDF17020B1B8ACD00F8391B /* stack_frame_cpu.h */,
9BDF17030B1B8ACD00F8391B /* stackwalker.h */,
9BDF17040B1B8ACD00F8391B /* symbol_supplier.h */,
+ 9B44619D0B66C66B00BBB817 /* system_info.h */,
);
path = processor;
sourceTree = "<group>";
diff --git a/src/tools/mac/crash_report/on_demand_symbol_supplier.h b/src/tools/mac/crash_report/on_demand_symbol_supplier.h
index 16ab7a49..0cd78f58 100644
--- a/src/tools/mac/crash_report/on_demand_symbol_supplier.h
+++ b/src/tools/mac/crash_report/on_demand_symbol_supplier.h
@@ -45,20 +45,17 @@ class MinidumpModule;
class OnDemandSymbolSupplier : public SymbolSupplier {
public:
- // |architecture| should be one of ppc, i386, x86, ppc64, x86_64
// |search_dir| is the directory to search for alternative symbols with
// the same name as the module in the minidump
- OnDemandSymbolSupplier(const string &architecture, const string &search_dir);
+ OnDemandSymbolSupplier(const string &search_dir);
virtual ~OnDemandSymbolSupplier() {}
// Returns the path to the symbol file for the given module.
- virtual SymbolResult GetSymbolFile(const CodeModule *module,
+ virtual SymbolResult GetSymbolFile(const CodeModule *module,
+ const SystemInfo *system_info,
string *symbol_file);
protected:
- // Return symbols for this architecture
- string architecture_;
-
// Search directory
string search_dir_;
@@ -84,7 +81,8 @@ class OnDemandSymbolSupplier : public SymbolSupplier {
// Generate the airbag symbol file for |module|. Return true if successful.
// File is generated in /tmp.
- bool GenerateSymbolFile(const CodeModule *module);
+ bool GenerateSymbolFile(const CodeModule *module,
+ const SystemInfo *system_info);
};
} // namespace google_airbag
diff --git a/src/tools/mac/crash_report/on_demand_symbol_supplier.mm b/src/tools/mac/crash_report/on_demand_symbol_supplier.mm
index d37313d7..d0dd7fb8 100644
--- a/src/tools/mac/crash_report/on_demand_symbol_supplier.mm
+++ b/src/tools/mac/crash_report/on_demand_symbol_supplier.mm
@@ -33,6 +33,7 @@
#include "google_airbag/processor/basic_source_line_resolver.h"
#include "google_airbag/processor/minidump.h"
+#include "google_airbag/processor/system_info.h"
#include "processor/pathname_stripper.h"
#include "on_demand_symbol_supplier.h"
@@ -44,28 +45,28 @@ using std::string;
using google_airbag::OnDemandSymbolSupplier;
using google_airbag::PathnameStripper;
using google_airbag::SymbolSupplier;
+using google_airbag::SystemInfo;
-OnDemandSymbolSupplier::OnDemandSymbolSupplier(const string &architecture,
- const string &search_dir)
- : architecture_(architecture),
- search_dir_(search_dir) {
+OnDemandSymbolSupplier::OnDemandSymbolSupplier(const string &search_dir)
+ : search_dir_(search_dir) {
}
-SymbolSupplier::SymbolResult
+SymbolSupplier::SymbolResult
OnDemandSymbolSupplier::GetSymbolFile(const CodeModule *module,
+ const SystemInfo *system_info,
string *symbol_file) {
string path(GetModuleSymbolFile(module));
if (path.empty()) {
- if (!GenerateSymbolFile(module))
+ if (!GenerateSymbolFile(module, system_info))
return NOT_FOUND;
-
+
path = GetModuleSymbolFile(module);
}
-
+
if (path.empty())
return NOT_FOUND;
-
+
*symbol_file = path;
return FOUND;
}
@@ -127,16 +128,17 @@ static float GetFileModificationTime(const char *path) {
if (stat(path, &file_stat) == 0)
result = (float)file_stat.st_mtimespec.tv_sec +
(float)file_stat.st_mtimespec.tv_nsec / 1.0e9;
-
+
return result;
}
-bool OnDemandSymbolSupplier::GenerateSymbolFile(const CodeModule *module) {
+bool OnDemandSymbolSupplier::GenerateSymbolFile(const CodeModule *module,
+ const SystemInfo *system_info) {
bool result = true;
string name = GetNameForModule(module);
string module_path = GetLocalModulePath(module);
NSString *symbol_path = [NSString stringWithFormat:@"/tmp/%s.%s.sym",
- name.c_str(), architecture_.c_str()];
+ name.c_str(), system_info->cpu.c_str()];
if (module_path.empty())
return false;
@@ -146,11 +148,11 @@ bool OnDemandSymbolSupplier::GenerateSymbolFile(const CodeModule *module) {
BOOL generate_file = YES;
if ([[NSFileManager defaultManager] fileExistsAtPath:symbol_path]) {
// Check if the module file is newer than the saved symbols
- float cache_time =
+ float cache_time =
GetFileModificationTime([symbol_path fileSystemRepresentation]);
- float module_time =
+ float module_time =
GetFileModificationTime(module_path.c_str());
-
+
if (cache_time > module_time)
generate_file = NO;
}
@@ -160,7 +162,7 @@ bool OnDemandSymbolSupplier::GenerateSymbolFile(const CodeModule *module) {
stringWithFileSystemRepresentation:module_path.c_str()
length:module_path.length()];
DumpSymbols *dump = [[DumpSymbols alloc] initWithContentsOfFile:module_str];
- const char *archStr = architecture_.c_str();
+ const char *archStr = system_info->cpu.c_str();
if ([dump setArchitecture:[NSString stringWithUTF8String:archStr]]) {
[dump writeSymbolFile:symbol_path];
} else {