yuzu-emu/breakpad
1
0
Fork 0
mirror of https://github.com/yuzu-emu/breakpad.git synced 2024-11-24 05:35:39 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix 'debug_file' in PESourceLineWriter.

Browse Source 
- Add a #define to testing.gyp to avoid warnings about TR1 deprecation.
- PESourceLineWriter now reads debug_file from CodeView record instead
  of using code_file value.
- Updated PE-only MD reading unit test.

Change-Id: Ib4e6201df3e3fd651e160f310584b5a67b16c842
Reviewed-on: https://chromium-review.googlesource.com/c/breakpad/breakpad/+/1668347
Reviewed-by: Ivan Penkov <ivanpe@chromium.org>
This commit is contained in:
Nelson Billing 2019-06-19 16:13:24 -07:00
parent ee65a3c005
commit c38725b70e
3 changed files with 29 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/client/windows/unittests/testing.gyp
View File

@ -53,9 +53,9 @@
# Visual C++ implements variadic templates strangely, and # Visual C++ implements variadic templates strangely, and
# VC++2012 broke Google Test by lowering this value. See # VC++2012 broke Google Test by lowering this value. See
# http://stackoverflow.com/questions/12558327/google-test-in-visual-studio-2012 # http://stackoverflow.com/questions/12558327/google-test-in-visual-studio-2012
'defines': ['_VARIADIC_MAX=10'], 'defines': ['_VARIADIC_MAX=10', '_SILENCE_TR1_NAMESPACE_DEPRECATION_WARNING'],
}, },
'defines': ['_VARIADIC_MAX=10'], 'defines': ['_VARIADIC_MAX=10', '_SILENCE_TR1_NAMESPACE_DEPRECATION_WARNING'],
}, },
{ {
'target_name': 'gmock', 'target_name': 'gmock',
@ -81,9 +81,9 @@
'<(DEPTH)/testing/googlemock', '<(DEPTH)/testing/googlemock',
'<(DEPTH)/testing', '<(DEPTH)/testing',
], ],
'defines': ['_VARIADIC_MAX=10'], 'defines': ['_VARIADIC_MAX=10', '_SILENCE_TR1_NAMESPACE_DEPRECATION_WARNING'],
}, },
'defines': ['_VARIADIC_MAX=10'], 'defines': ['_VARIADIC_MAX=10', '_SILENCE_TR1_NAMESPACE_DEPRECATION_WARNING'],
}, },
], ],

34
src/common/windows/pe_util.cc
View File

@ -95,7 +95,7 @@ struct CV_INFO_PDB70 {
ULONG cv_signature; ULONG cv_signature;
GUID signature; GUID signature;
ULONG age; ULONG age;
UCHAR pdb_filename[ANYSIZE_ARRAY]; CHAR pdb_filename[ANYSIZE_ARRAY];
}; };
#define CV_SIGNATURE_RSDS 'SDSR' #define CV_SIGNATURE_RSDS 'SDSR'
@ -123,8 +123,6 @@ using std::unique_ptr;
using google_breakpad::GUIDString; using google_breakpad::GUIDString;
bool ReadModuleInfo(const wstring & pe_file, PDBModuleInfo * info) { bool ReadModuleInfo(const wstring & pe_file, PDBModuleInfo * info) {
info->debug_file = WindowsStringUtils::GetBaseName(pe_file);
// Convert wchar to native charset because ImageLoad only takes // Convert wchar to native charset because ImageLoad only takes
// a PSTR as input. // a PSTR as input.
string img_file; string img_file;
@ -141,10 +139,10 @@ bool ReadModuleInfo(const wstring & pe_file, PDBModuleInfo * info) {
} }
info->cpu = FileHeaderMachineToCpuString( info->cpu = FileHeaderMachineToCpuString(
img->FileHeader->FileHeader.Machine); img->FileHeader->FileHeader.Machine);
PIMAGE_OPTIONAL_HEADER64 optional_header = PIMAGE_OPTIONAL_HEADER64 optional_header =
&(reinterpret_cast<PIMAGE_NT_HEADERS64>(img->FileHeader))->OptionalHeader; &(reinterpret_cast<PIMAGE_NT_HEADERS64>(img->FileHeader))->OptionalHeader;
if (optional_header->Magic != IMAGE_NT_OPTIONAL_HDR64_MAGIC) { if (optional_header->Magic != IMAGE_NT_OPTIONAL_HDR64_MAGIC) {
fprintf(stderr, "Not a PE32+ image\n"); fprintf(stderr, "Not a PE32+ image\n");
return false; return false;
@ -164,21 +162,37 @@ bool ReadModuleInfo(const wstring & pe_file, PDBModuleInfo * info) {
for (DWORD i = 0; i < debug_size / sizeof(*debug_directories); i++) { for (DWORD i = 0; i < debug_size / sizeof(*debug_directories); i++) {
if (debug_directories[i].Type != IMAGE_DEBUG_TYPE_CODEVIEW || if (debug_directories[i].Type != IMAGE_DEBUG_TYPE_CODEVIEW ||
debug_directories[i].SizeOfData < sizeof(CV_INFO_PDB70)) { debug_directories[i].SizeOfData < sizeof(CV_INFO_PDB70)) {
continue; continue;
} }
struct CV_INFO_PDB70* cv_info = static_cast<CV_INFO_PDB70*>(ImageRvaToVa( struct CV_INFO_PDB70* cv_info = static_cast<CV_INFO_PDB70*>(ImageRvaToVa(
img->FileHeader, img->FileHeader,
img->MappedAddress, img->MappedAddress,
debug_directories[i].AddressOfRawData, debug_directories[i].AddressOfRawData,
&img->LastRvaSection)); &img->LastRvaSection));
if (cv_info->cv_signature != CV_SIGNATURE_RSDS) { if (cv_info->cv_signature != CV_SIGNATURE_RSDS) {
continue; continue;
} }
info->debug_identifier = GenerateDebugIdentifier(cv_info->age, info->debug_identifier = GenerateDebugIdentifier(cv_info->age,
cv_info->signature); cv_info->signature);
// This code assumes that the pdb_filename is stored as ASCII without
// multibyte characters, but it's not clear if that's true.
size_t debug_file_length = strnlen_s(cv_info->pdb_filename, MAX_PATH);
if (debug_file_length < 0 || debug_file_length >= MAX_PATH) {
fprintf(stderr, "PE debug directory is corrupt.\n");
return false;
}
std::string debug_file(cv_info->pdb_filename, debug_file_length);
if (!WindowsStringUtils::safe_mbstowcs(debug_file, &info->debug_file)) {
fprintf(stderr, "PDB filename '%s' contains unrecognized characters.\n",
debug_file.c_str());
return false;
}
info->debug_file = WindowsStringUtils::GetBaseName(info->debug_file);
return true; return true;
} }

2
src/tools/windows/dump_syms/testdata/pe_only_symbol_test.sym vendored
View File

@ -1,4 +1,4 @@
MODULE windows x86_64 2A5EAB481FAB4A17A9761CDC14FE531A1 pe_only_symbol_test.dll MODULE windows x86_64 2A5EAB481FAB4A17A9761CDC14FE531A1 pe_only_symbol_test.pdb
INFO CODE_ID 5C8AD05F12000 pe_only_symbol_test.dll INFO CODE_ID 5C8AD05F12000 pe_only_symbol_test.dll
STACK CFI INIT 1440 39 .cfa: $rsp .ra: .cfa 8 - ^ STACK CFI INIT 1440 39 .cfa: $rsp .ra: .cfa 8 - ^
STACK CFI 1440 .cfa: $rsp 32 + STACK CFI 1440 .cfa: $rsp 32 +