[Lldb-commits] [lldb] r368838 - Revert "Minidump/Windows: Fix module lookup"

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Wed Aug 14 05:26:51 PDT 2019


Author: labath
Date: Wed Aug 14 05:26:51 2019
New Revision: 368838

URL: http://llvm.org/viewvc/llvm-project?rev=368838&view=rev
Log:
Revert "Minidump/Windows: Fix module lookup"

Although there is nothing wrong with this patch, the test added here
uncovers a problem in other parts of the code which cause the test to
fail when running under asan. Reverting the patch until I can fix the
underlying issue(s).

This reverts commit r368416.

Removed:
    lldb/trunk/lit/Minidump/Windows/Inputs/find-module.dmp.yaml
    lldb/trunk/lit/Minidump/Windows/Inputs/find-module.exe.yaml
    lldb/trunk/lit/Minidump/Windows/find-module.test
Modified:
    lldb/trunk/source/Plugins/Process/minidump/MinidumpParser.cpp

Removed: lldb/trunk/lit/Minidump/Windows/Inputs/find-module.dmp.yaml
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/lit/Minidump/Windows/Inputs/find-module.dmp.yaml?rev=368837&view=auto
==============================================================================
--- lldb/trunk/lit/Minidump/Windows/Inputs/find-module.dmp.yaml (original)
+++ lldb/trunk/lit/Minidump/Windows/Inputs/find-module.dmp.yaml (removed)
@@ -1,26 +0,0 @@
---- !minidump
-Streams:
-  - Type:            ModuleList
-    Modules:
-      - Base of Image:   0x00000000000B0000
-        Size of Image:   0x00005000
-        Module Name:     'find-module.exe'
-        CodeView Record: 525344533ED87D89C8A8184197F3A925EE4BF74101000000433A5C70726F6A656374735C746573745F6170705C436F6E736F6C654170706C69636174696F6E315C44656275675C436F6E736F6C654170706C69636174696F6E312E70646200
-  - Type:            SystemInfo
-    Processor Arch:  X86
-    Processor Level: 23
-    Processor Revision: 2050
-    Number of Processors: 32
-    Product type:    1
-    Major Version:   10
-    Build Number:    17134
-    Platform ID:     Win32NT
-    Suite Mask:      0x0300
-    CPU:
-      Vendor ID:       AuthenticAMD
-      Version Info:    0x00800F82
-      Feature Info:    0x178BFBFF
-      AMD Extended Features: 0x2FD3FBFF
-  - Type:            MiscInfo
-    Content:         54050000F703000008290000C883495D0000000000000000AC0D000098080000AC0D00000200000002000000002000000D0000000000000002000000C4FFFFFF430065006E007400720061006C0020004500750072006F007000650020005300740061006E0064006100720064002000540069006D006500000000000000000000000A0000000500030000000000000000000000430065006E007400720061006C0020004500750072006F00700065002000530075006D006D00650072002000540069006D00650000000000000000000000000000000300000005000200000000000000C4FFFFFF310037003100330034002E0031002E007800380036006600720065002E007200730034005F00720065006C0065006100730065002E003100380030003400310030002D00310038003000340000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000064006200670063006F00720065002E0069003300380036002C00310030002E0030002E00310037003100330034002E0031000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000006C816B92
-...

Removed: lldb/trunk/lit/Minidump/Windows/Inputs/find-module.exe.yaml
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/lit/Minidump/Windows/Inputs/find-module.exe.yaml?rev=368837&view=auto
==============================================================================
--- lldb/trunk/lit/Minidump/Windows/Inputs/find-module.exe.yaml (original)
+++ lldb/trunk/lit/Minidump/Windows/Inputs/find-module.exe.yaml (removed)
@@ -1,32 +0,0 @@
---- !COFF
-OptionalHeader:
-  AddressOfEntryPoint: 4224
-  ImageBase:       4194304
-  SectionAlignment: 4096
-  FileAlignment:   512
-  MajorOperatingSystemVersion: 6
-  MinorOperatingSystemVersion: 0
-  MajorImageVersion: 0
-  MinorImageVersion: 0
-  MajorSubsystemVersion: 6
-  MinorSubsystemVersion: 0
-  Subsystem:       IMAGE_SUBSYSTEM_WINDOWS_CUI
-  DLLCharacteristics: [ IMAGE_DLL_CHARACTERISTICS_DYNAMIC_BASE, IMAGE_DLL_CHARACTERISTICS_NX_COMPAT, IMAGE_DLL_CHARACTERISTICS_NO_SEH, IMAGE_DLL_CHARACTERISTICS_TERMINAL_SERVER_AWARE ]
-  SizeOfStackReserve: 1048576
-  SizeOfStackCommit: 4096
-  SizeOfHeapReserve: 1048576
-  SizeOfHeapCommit: 4096
-header:
-  Machine:         IMAGE_FILE_MACHINE_I386
-  Characteristics: [ IMAGE_FILE_EXECUTABLE_IMAGE, IMAGE_FILE_32BIT_MACHINE ]
-sections:
-  - Name:            .rdata
-    Characteristics: [ IMAGE_SCN_CNT_INITIALIZED_DATA, IMAGE_SCN_MEM_READ ]
-    VirtualAddress:  8192
-    VirtualSize:     304
-  - Name:            .reloc
-    Characteristics: [ IMAGE_SCN_CNT_INITIALIZED_DATA, IMAGE_SCN_MEM_DISCARDABLE, IMAGE_SCN_MEM_READ ]
-    VirtualAddress:  16384
-    VirtualSize:     48
-symbols:         []
-...

Removed: lldb/trunk/lit/Minidump/Windows/find-module.test
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/lit/Minidump/Windows/find-module.test?rev=368837&view=auto
==============================================================================
--- lldb/trunk/lit/Minidump/Windows/find-module.test (original)
+++ lldb/trunk/lit/Minidump/Windows/find-module.test (removed)
@@ -1,10 +0,0 @@
-Test that we correctly find a PE/COFF file in our executable search path, and
-use it when opening minidumps.
-
-RUN: yaml2obj %S/Inputs/find-module.exe.yaml > %T/find-module.exe
-RUN: yaml2obj %S/Inputs/find-module.dmp.yaml > %T/find-module.dmp
-RUN: %lldb -O "settings set target.exec-search-paths %T" \
-RUN:   -c %T/find-module.dmp -o "image dump objfile" -o exit | FileCheck %s
-
-CHECK-LABEL: image dump objfile
-CHECK: ObjectFilePECOFF, file = '{{.*}}find-module.exe', arch = i386

Modified: lldb/trunk/source/Plugins/Process/minidump/MinidumpParser.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Process/minidump/MinidumpParser.cpp?rev=368838&r1=368837&r2=368838&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Process/minidump/MinidumpParser.cpp (original)
+++ lldb/trunk/source/Plugins/Process/minidump/MinidumpParser.cpp Wed Aug 14 05:26:51 2019
@@ -188,7 +188,6 @@ ArchSpec MinidumpParser::GetArchitecture
   case OSPlatform::Win32NT:
   case OSPlatform::Win32CE:
     triple.setOS(llvm::Triple::OSType::Win32);
-    triple.setVendor(llvm::Triple::VendorType::PC);
     break;
   case OSPlatform::Linux:
     triple.setOS(llvm::Triple::OSType::Linux);




More information about the lldb-commits mailing list