From 7837cb7236528b7fe0a35ada0c231cbef9a2e6aa Mon Sep 17 00:00:00 2001 From: "ted.mielczarek" Date: Wed, 3 Dec 2008 18:23:09 +0000 Subject: [PATCH] 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 --- src/common/mac/dump_syms.h | 1 - src/common/mac/dump_syms.mm | 100 ++++++------------------------------ 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 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:)];