[PATCH 00/10] Beefing up tests for dbghelp (v2)

Eric Pouech eric.pouech at gmail.com
Thu Jul 15 02:38:51 CDT 2021


The following series beefs up the tests about dbghelp DLL and fixes a
couple of bugs in dbghelp.

The idea is to have a set of files to get debug info from (debuggee*.*).

This is supported in two different ways:
A) to be compiled directly inside dbghelp_test: the default tools and options
   used in Wine will be covered (now: cross compile + dwarf version 2 debug
   format) 
B) but also the ability to run the same tests on an external process + image
   (made mainly out of the debuggee*.* only files). The idea is of course
   to use different compilers, debug format, options, 32/64bit targets...
   Those alternate images **ARE NOT** part (for now) of winetest.

Unfortunately, with Winetest Bot, the A) won't work under windows: debug
format of dbghelp_test.exe is dwarf2, while native dbghelp doesn't support
it. So the tests are skipped. Anyway, most of the Windows setup:s don't
have native dbghelp.dll installed <g>, so that shouldn't change much.

I locally test every patch with:
- A) on 32bit Wine (mingw + dwarf2)
- A) on 64bit Wine (mingw + dwarf2)
- A) on Windows10 as PE64 (msc + pdb)
- B) image: mingw + dwarf2 + target=PE64bit + option -O1 on 64bit Wine
- B) image: mingw + dwarf2 + target=PE32bit + option -O1 on 32bit Wine
- B) image: ms compiler + pdb + target=PE64bit + option /O1 on 64 bit Wine

This serie will be continued with:
1) more tests (and more fixes) (around 20+ patches to come)
2) support of dwarf3 and 4 (around 70+ patches to come)

I also plan to add more local tests (dwarf3 and dwarf4; WoW; PE32 + pdb on
Wine, cross 32/64 bit between tester and debuggee, other options O2).

Changes introduced in v2:
- make sure that dbghelp_test.exe embedded as resource inside winetest.exe
  isn't stripped, so that on can access its debug information.
    tools/makedep.c is fixed accordingly.
  This should fix all the TestBot failures.
- the existing tests (before this series) for dbghelp, could somehow work
  when no debug information is available. Make sure we don't change this
  behavior (v1 was skipping those tests when no debug info is avail)

A+

---

Eric Pouech (10):
      tools/makedep: added support for not stripping some test modules
      dbghelp/tests: Added dedicated structure to control the debuggee
      dbghelp/tests: Added ability to launch an external process instead of debugging self
      dbghelp/tests: Now properly getting base of main module inside debuggee structure
      dbghelp/tests: Added two new compilation units (debuggee1.c and debuggee2.c)
      dbghelp/tests: now building inside each debuggee a manifest
      dbghelp/tests: StackWalk64 testing now support external process
      dbghelp/tests: Added tests about getting symbol information
      dbghelp: returns name of compiland in SymGetTypeInfo
      dbghelp: handling of complex location in dwarf


 dlls/dbghelp/dwarf.c           |   1 -
 dlls/dbghelp/tests/Makefile.in |   4 +-
 dlls/dbghelp/tests/dbghelp.c   | 547 +++++++++++++++++++++++++++++++--
 dlls/dbghelp/tests/debuggee.h  |  30 ++
 dlls/dbghelp/tests/debuggee1.c | 115 +++++++
 dlls/dbghelp/tests/debuggee2.c |   7 +
 dlls/dbghelp/type.c            |   3 +-
 tools/makedep.c                |  11 +-
 8 files changed, 673 insertions(+), 45 deletions(-)
 create mode 100644 dlls/dbghelp/tests/debuggee.h
 create mode 100644 dlls/dbghelp/tests/debuggee1.c
 create mode 100644 dlls/dbghelp/tests/debuggee2.c




More information about the wine-devel mailing list