aboutsummaryrefslogtreecommitdiff
path: root/src/common
diff options
context:
space:
mode:
authorted.mielczarek <ted.mielczarek@4c0a9323-5329-0410-9bdc-e9ce6186880e>2008-12-03 18:23:09 +0000
committerted.mielczarek <ted.mielczarek@4c0a9323-5329-0410-9bdc-e9ce6186880e>2008-12-03 18:23:09 +0000
commit7837cb7236528b7fe0a35ada0c231cbef9a2e6aa (patch)
tree3655634cfcf6099d0cbbb5932c739d09b85613ce /src/common
parentExceptionHandler class is not handling initialization errors, such as the (diff)
downloadbreakpad-7837cb7236528b7fe0a35ada0c231cbef9a2e6aa.tar.xz
issue 286 - clean up some demangling code in dump_syms.mm. r=chris rogers
git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@301 4c0a9323-5329-0410-9bdc-e9ce6186880e
Diffstat (limited to 'src/common')
-rw-r--r--src/common/mac/dump_syms.h1
-rw-r--r--src/common/mac/dump_syms.mm100
2 files changed, 16 insertions, 85 deletions
diff --git a/src/common/mac/dump_syms.h b/src/common/mac/dump_syms.h
index e234a6a6..12d8f7d6 100644
--- a/src/common/mac/dump_syms.h
+++ b/src/common/mac/dump_syms.h
@@ -46,7 +46,6 @@ typedef hash_map<string, dwarf2reader::SectionMap *> ArchSectionMap;
NSMutableDictionary *addresses_; // Addresses and symbols (STRONG)
NSMutableSet *functionAddresses_; // Function addresses (STRONG)
NSMutableDictionary *sources_; // Address and Source file paths (STRONG)
- NSMutableArray *cppAddresses_; // Addresses of C++ symbols (STRONG)
NSMutableDictionary *headers_; // Mach-o header information (STRONG)
NSMutableDictionary *sectionData_; // Keyed by seg/sect name (STRONG)
uint32_t lastStartAddress_;
diff --git a/src/common/mac/dump_syms.mm b/src/common/mac/dump_syms.mm
index b4f779df..901f6276 100644
--- a/src/common/mac/dump_syms.mm
+++ b/src/common/mac/dump_syms.mm
@@ -104,8 +104,7 @@ void DumpFunctionMap(const dwarf2reader::FunctionMap function_map) {
@interface DumpSymbols(PrivateMethods)
-- (NSArray *)convertCPlusPlusSymbols:(NSArray *)symbols;
-- (void)convertSymbols;
+- (NSString *)convertCPlusPlusSymbol:(NSString *)symbol;
- (void)addFunction:(NSString *)name line:(int)line address:(uint64_t)address section:(int)section;
- (BOOL)processSymbolItem:(struct nlist_64 *)list stringTable:(char *)table;
- (BOOL)loadSymbolInfo:(void *)base offset:(uint32_t)offset;
@@ -126,86 +125,20 @@ void DumpFunctionMap(const dwarf2reader::FunctionMap function_map) {
@implementation DumpSymbols
//=============================================================================
-- (NSArray *)convertCPlusPlusSymbols:(NSArray *)symbols {
- NSMutableArray *symbols_demangled = [[NSMutableArray alloc]
- initWithCapacity:[symbols count]];
+- (NSString *)convertCPlusPlusSymbol:(NSString *)symbol {
// __cxa_demangle will realloc this if needed
char *buffer = (char *)malloc(1024);
size_t buffer_size = 1024;
int result;
- NSEnumerator *enumerator = [symbols objectEnumerator];
- id symbolObject;
- while ((symbolObject = [enumerator nextObject])) {
- const char *symbol = [symbolObject UTF8String];
- buffer = abi::__cxa_demangle(symbol, buffer, &buffer_size, &result);
- if (result == 0) {
- [symbols_demangled addObject:[NSString stringWithUTF8String:buffer]];
- } else {
- // unable to demangle - use mangled name instead
- [symbols_demangled addObject:symbolObject];
- }
+ const char *sym = [symbol UTF8String];
+ NSString *demangled = nil;
+ buffer = abi::__cxa_demangle(sym, buffer, &buffer_size, &result);
+ if (result == 0) {
+ demangled = [NSString stringWithUTF8String:buffer];
}
free(buffer);
-
- return symbols_demangled;
-}
-
-//=============================================================================
-- (void)convertSymbols {
- unsigned int count = [cppAddresses_ count];
- NSMutableArray *symbols = [[NSMutableArray alloc] initWithCapacity:count];
-
- // Sort addresses for processing
- NSArray *addresses = [cppAddresses_ sortedArrayUsingSelector:
- @selector(compare:)];
-
- for (unsigned int i = 0; i < count; ++i) {
- NSMutableDictionary *dict = [addresses_ objectForKey:
- [addresses objectAtIndex:i]];
- NSString *symbol = [dict objectForKey:kAddressSymbolKey];
-
- // Make sure that the symbol is valid
- if ([symbol length] < 1)
- symbol = kUnknownSymbol;
-
- [symbols addObject:symbol];
- }
-
- // In order to deal with crashing problems in c++filt, we setup
- // a while loop to handle the case where convertCPlusPlusSymbols
- // only returns partial results.
- // We then attempt to continue from the point where c++filt failed
- // and add the partial results to the total results until we're
- // completely done.
-
- unsigned int totalIndex = 0;
- unsigned int totalCount = count;
-
- while (totalIndex < totalCount) {
- NSRange range = NSMakeRange(totalIndex, totalCount - totalIndex);
- NSArray *subarray = [symbols subarrayWithRange:range];
- NSArray *converted = [self convertCPlusPlusSymbols:subarray];
- unsigned int convertedCount = [converted count];
-
- if (convertedCount == 0) {
- break; // we give up at this point
- }
-
- for (unsigned int convertedIndex = 0;
- convertedIndex < convertedCount && totalIndex < totalCount;
- ++totalIndex, ++convertedIndex) {
- NSMutableDictionary *dict = [addresses_ objectForKey:
- [addresses objectAtIndex:totalIndex]];
- NSString *symbol = [converted objectAtIndex:convertedIndex];
-
- // Only add if this is a non-zero length symbol
- if ([symbol length])
- [dict setObject:symbol forKey:kAddressConvertedSymbolKey];
- }
- }
-
- [symbols release];
+ return demangled;
}
//=============================================================================
@@ -215,8 +148,7 @@ void DumpFunctionMap(const dwarf2reader::FunctionMap function_map) {
if (!address)
return;
- // If the function starts with "_Z" or "__Z" then add it to the list of
- // addresses to run through the c++filt
+ // If the function starts with "_Z" or "__Z" then demangle it.
BOOL isCPP = NO;
if ([name hasPrefix:@"__Z"]) {
@@ -255,12 +187,6 @@ void DumpFunctionMap(const dwarf2reader::FunctionMap function_map) {
NSRange emptyRange = { NSNotFound, 0 };
NSRange objcppRange = [name rangeOfCharacterFromSet:objcppCharSet];
isCPP = NSEqualRanges(objcppRange, emptyRange);
- }
-
- if (isCPP) {
- if (!cppAddresses_)
- cppAddresses_ = [[NSMutableArray alloc] init];
- [cppAddresses_ addObject:addressNum];
} else if ([name characterAtIndex:0] == '_') {
// Remove the leading underscore
name = [name substringFromIndex:1];
@@ -282,6 +208,13 @@ void DumpFunctionMap(const dwarf2reader::FunctionMap function_map) {
// only functions, not line number addresses
[functionAddresses_ addObject:addressNum];
}
+
+ if (isCPP) {
+ // try demangling
+ NSString *demangled = [self convertCPlusPlusSymbol:name];
+ if (demangled != nil)
+ [dict setObject:demangled forKey:kAddressConvertedSymbolKey];
+ }
if (line && ![dict objectForKey:kAddressSourceLineKey])
[dict setObject:[NSNumber numberWithUnsignedInt:line]
@@ -920,7 +853,6 @@ static BOOL WriteFormat(int fd, const char *fmt, ...) {
// Gather the information
[self loadSymbolInfoForArchitecture];
- [self convertSymbols];
NSArray *sortedAddresses = [[addresses_ allKeys]
sortedArrayUsingSelector:@selector(compare:)];