maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Fixed an issue where we might have easy access to the string table data for a mach file from memory even though we have a process. So now we don't read the string table strings from memory when we don't have to. 

llvm-svn: 157482
This commit is contained in:
Greg Clayton 2012-05-25 17:04:00 +00:00
parent a6da3ff896
commit debb881079
1 changed files with 21 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
lldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp
View File

@ -1359,6 +1359,9 @@ ObjectFileMachO::ParseSymtab (bool minimize)
}
const bool have_strtab_data = strtab_data.GetByteSize() > 0;
if (!have_strtab_data)
{
if (process)
{
if (strtab_addr == LLDB_INVALID_ADDRESS)
@ -1368,12 +1371,13 @@ ObjectFileMachO::ParseSymtab (bool minimize)
return 0;
}
}
else if (strtab_data.GetByteSize() == 0)
else
{
if (log)
module_sp->LogMessage(log.get(), "failed to read strtab data");
return 0;
}
}
const ConstString &g_segment_name_TEXT = GetSegmentNameTEXT();
const ConstString &g_segment_name_DATA = GetSegmentNameDATA();
@ -1449,14 +1453,7 @@ ObjectFileMachO::ParseSymtab (bool minimize)
SymbolType type = eSymbolTypeInvalid;
const char *symbol_name = NULL;
if (process)
{
const addr_t str_addr = strtab_addr + nlist.n_strx;
Error str_error;
if (process->ReadCStringFromMemory(str_addr, memory_symbol_name, str_error))
symbol_name = memory_symbol_name.c_str();
}
else
if (have_strtab_data)
{
symbol_name = strtab_data.PeekCStr(nlist.n_strx);
@ -1476,6 +1473,13 @@ ObjectFileMachO::ParseSymtab (bool minimize)
if (symbol_name[0] == '\0')
symbol_name = NULL;
}
else
{
const addr_t str_addr = strtab_addr + nlist.n_strx;
Error str_error;
if (process->ReadCStringFromMemory(str_addr, memory_symbol_name, str_error))
symbol_name = memory_symbol_name.c_str();
}
const char *symbol_name_non_abi_mangled = NULL;
SectionSP symbol_section;