December 2009 - Wine-Commits

Eric Pouech : winedbg: For the x86_64 backend, fetch all the integral types .
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: 91c7ff299469a07c37fc10ea681f90d2ddfaaccb URL: http://source.winehq.org/git/wine.git/?a=commit;h=91c7ff299469a07c37fc10ea6… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Dec 12 12:19:38 2009 +0100 winedbg: For the x86_64 backend, fetch all the integral types. --- programs/winedbg/be_x86_64.c | 20 +++++++++++++++++--- 1 files changed, 17 insertions(+), 3 deletions(-) diff --git a/programs/winedbg/be_x86_64.c b/programs/winedbg/be_x86_64.c index dc1f227..ed5cdf2 100644 --- a/programs/winedbg/be_x86_64.c +++ b/programs/winedbg/be_x86_64.c @@ -226,10 +226,24 @@ static int be_x86_64_adjust_pc_for_break(CONTEXT* ctx, BOOL way) } static int be_x86_64_fetch_integer(const struct dbg_lvalue* lvalue, unsigned size, - unsigned ext_sign, LONGLONG* ret) + unsigned ext_sign, LONGLONG* ret) { - dbg_printf("not done fetch_integer\n"); - return FALSE; + if (size != 1 && size != 2 && size != 4 && size != 8 && size != 16) + return FALSE; + + memset(ret, 0, sizeof(*ret)); /* clear unread bytes */ + /* FIXME: this assumes that debuggee and debugger use the same + * integral representation + */ + if (!memory_read_value(lvalue, size, ret)) return FALSE; + + /* propagate sign information */ + if (ext_sign && size < 16 && (*ret >> (size * 8 - 1)) != 0) + { + ULONGLONG neg = -1; + *ret |= neg << (size * 8); + } + return TRUE; } static int be_x86_64_fetch_float(const struct dbg_lvalue* lvalue, unsigned size,

1 0

Eric Pouech : winedbg: Force 64bit line number information throughout the code to ease 64bit portability .
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: 12c3a012cbe1ecaa7cd7369ef09b48b241b01b32 URL: http://source.winehq.org/git/wine.git/?a=commit;h=12c3a012cbe1ecaa7cd7369ef… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Dec 12 12:19:44 2009 +0100 winedbg: Force 64bit line number information throughout the code to ease 64bit portability. --- programs/winedbg/break.c | 4 ++-- programs/winedbg/dbg.y | 4 ++-- programs/winedbg/debugger.h | 4 ++-- programs/winedbg/memory.c | 4 ++-- programs/winedbg/source.c | 10 +++++----- programs/winedbg/stack.c | 6 +++--- programs/winedbg/symbol.c | 21 +++++++++++---------- programs/winedbg/tgt_active.c | 4 ++-- 8 files changed, 29 insertions(+), 28 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=12c3a012cbe1ecaa7cd73…

1 0

Hans Leidekker : rpcrt4: Add tests for RPC calls with authentication.
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: fddabccaa1daae17f5fde0060d02d36b80acb080 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fddabccaa1daae17f5fde0060… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Mon Dec 14 11:54:36 2009 +0100 rpcrt4: Add tests for RPC calls with authentication. --- dlls/rpcrt4/tests/Makefile.in | 2 +- dlls/rpcrt4/tests/server.c | 159 ++++++++++++++++++++++++++++++++++++----- dlls/rpcrt4/tests/server.idl | 2 + 3 files changed, 145 insertions(+), 18 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=fddabccaa1daae17f5fde…

1 0

Maarten Lankhorst : mmdevapi: Add more tests.
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: 0c8fbb288f285a8083bc3693291cd18b137df170 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0c8fbb288f285a8083bc36932… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Mon Dec 14 10:09:35 2009 +0100 mmdevapi: Add more tests. --- dlls/mmdevapi/tests/Makefile.in | 1 + dlls/mmdevapi/tests/dependency.c | 101 ++++++++++++++++++++++++++++++++++++++ 2 files changed, 102 insertions(+), 0 deletions(-) diff --git a/dlls/mmdevapi/tests/Makefile.in b/dlls/mmdevapi/tests/Makefile.in index 4ac467c..ef58c63 100644 --- a/dlls/mmdevapi/tests/Makefile.in +++ b/dlls/mmdevapi/tests/Makefile.in @@ -6,6 +6,7 @@ TESTDLL = mmdevapi.dll IMPORTS = ole32 version user32 kernel32 CTESTS = \ + dependency.c \ mmdevenum.c @MAKE_TEST_RULES@ diff --git a/dlls/mmdevapi/tests/dependency.c b/dlls/mmdevapi/tests/dependency.c new file mode 100644 index 0000000..1568bed --- /dev/null +++ b/dlls/mmdevapi/tests/dependency.c @@ -0,0 +1,101 @@ +/* + * Copyright 2009 Maarten Lankhorst + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include "wine/test.h" + +#define CINTERFACE +#define COBJMACROS + +#ifdef STANDALONE +#include "initguid.h" +#endif + +#include "unknwn.h" +#include "uuids.h" +#include "mmdeviceapi.h" +#include "dshow.h" +#include "dsound.h" + +static inline const char *dump_guid( const GUID *id ) +{ + static char data[39]; + if (!id) return "(null)"; + wsprintfA( data, "{%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x}", + id->Data1, id->Data2, id->Data3, + id->Data4[0], id->Data4[1], id->Data4[2], id->Data4[3], + id->Data4[4], id->Data4[5], id->Data4[6], id->Data4[7] ); + return data; +} + +START_TEST(dependency) +{ + HRESULT hr; + IMMDeviceEnumerator *mme = NULL; + IMMDevice *dev = NULL; + IDirectSound8 *ds8 = NULL; + IBaseFilter *bf = NULL; + + CoInitializeEx(NULL, COINIT_MULTITHREADED); + hr = CoCreateInstance(&CLSID_MMDeviceEnumerator, NULL, CLSCTX_INPROC_SERVER, &IID_IMMDeviceEnumerator, (void**)&mme); + if (FAILED(hr)) + { + skip("mmdevapi not available: 0x%08x\n", hr); + return; + } + + hr = IMMDeviceEnumerator_GetDefaultAudioEndpoint(mme, eRender, eMultimedia, &dev); + ok(hr == S_OK, "GetDefaultAudioEndpoint failed: 0x%08x\n", hr); + if (hr != S_OK) + { + skip("GetDefaultAudioEndpoint returns 0x%08x, skipping tests\n", hr); + return; + } + + ok(!GetModuleHandle("dsound.dll"), "dsound.dll was already loaded!\n"); + + hr = IMMDevice_Activate(dev, &IID_IDirectSound8, CLSCTX_INPROC_SERVER, NULL, (void**)&ds8); + ok(hr == S_OK, "Activating ds8 interface failed: 0x%08x\n", hr); + if (hr == S_OK) + { + ok(GetModuleHandle("dsound.dll") != NULL, "dsound.dll not loaded!\n"); + ok(ds8 != NULL, "ds8 pointer is null\n"); + } + if (ds8) + IUnknown_Release(ds8); + + ok(!GetModuleHandle("quartz.dll"), "quartz.dll was already loaded!\n"); + hr = IMMDevice_Activate(dev, &IID_IBaseFilter, CLSCTX_INPROC_SERVER, NULL, (void**)&bf); + ok(hr == S_OK, "Activating bf failed: 0x%08x\n", hr); + if (hr == S_OK) + { + ok(GetModuleHandle("quartz.dll") != NULL, "quartz.dll not loaded!\n"); + ok(bf != NULL, "bf pointer is null\n"); + if (bf) + { + CLSID clsid; + hr = IBaseFilter_GetClassID(bf, &clsid); + ok(hr == S_OK, "GetClassId failed with 0x%08x\n", hr); + if (hr == S_OK) + ok(IsEqualCLSID(&clsid, &CLSID_DSoundRender), "Wrong class id %s", dump_guid(&clsid)); + } + } + if (bf) + IUnknown_Release(bf); + + IUnknown_Release(mme); +}

1 0

Eric Pouech : winedbg: Added a couple of tweaks to properly support 32bit vs 64bit wide addresses .
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: ab2e0bdef6a6bd4718f4198ce21d33e176e6def9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ab2e0bdef6a6bd4718f4198ce… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Dec 12 12:19:29 2009 +0100 winedbg: Added a couple of tweaks to properly support 32bit vs 64bit wide addresses. --- programs/winedbg/debugger.h | 3 +++ programs/winedbg/gdbproxy.c | 5 +++-- programs/winedbg/info.c | 22 ++++++++++++---------- 3 files changed, 18 insertions(+), 12 deletions(-) diff --git a/programs/winedbg/debugger.h b/programs/winedbg/debugger.h index 30846eb..c02b71e 100644 --- a/programs/winedbg/debugger.h +++ b/programs/winedbg/debugger.h @@ -38,6 +38,9 @@ #include "objbase.h" #include "oaidl.h" +#define ADDRSIZE ((unsigned)sizeof(void*)) +#define ADDRWIDTH (ADDRSIZE * 2) + /* the debugger uses these exceptions for it's internal use */ #define DEBUG_STATUS_OFFSET 0x80003000 #define DEBUG_STATUS_INTERNAL_ERROR (DEBUG_STATUS_OFFSET+0) /* something went wrong */ diff --git a/programs/winedbg/gdbproxy.c b/programs/winedbg/gdbproxy.c index 65b1cc3..ceae777 100644 --- a/programs/winedbg/gdbproxy.c +++ b/programs/winedbg/gdbproxy.c @@ -1510,10 +1510,11 @@ static void packet_query_monitor_wnd_helper(struct gdb_context* gdbctx, HWND hWn packet_reply_open(gdbctx); packet_reply_catc(gdbctx, 'O'); snprintf(buffer, sizeof(buffer), - "%*s%04lx%*s%-17.17s %08x %08lx %.14s\n", + "%*s%04lx%*s%-17.17s %08x %0*lx %.14s\n", indent, "", (ULONG_PTR)hWnd, 13 - indent, "", clsName, GetWindowLongW(hWnd, GWL_STYLE), - (ULONG_PTR)GetWindowLongPtrW(hWnd, GWLP_WNDPROC), wndName); + ADDRWIDTH, (ULONG_PTR)GetWindowLongPtrW(hWnd, GWLP_WNDPROC), + wndName); packet_reply_hex_to_str(gdbctx, buffer); packet_reply_close(gdbctx); diff --git a/programs/winedbg/info.c b/programs/winedbg/info.c index 1b7eeb3..2afdf75 100644 --- a/programs/winedbg/info.c +++ b/programs/winedbg/info.c @@ -152,9 +152,9 @@ struct info_module static void module_print_info(const IMAGEHLP_MODULE64* mi, BOOL is_embedded) { - dbg_printf("%8s-%8s\t%-16s%s\n", - wine_dbgstr_longlong(mi->BaseOfImage), - wine_dbgstr_longlong(mi->BaseOfImage + mi->ImageSize), + dbg_printf("%*.*s-%*.*s\t%-16s%s\n", + ADDRWIDTH, ADDRWIDTH, wine_dbgstr_longlong(mi->BaseOfImage), + ADDRWIDTH, ADDRWIDTH, wine_dbgstr_longlong(mi->BaseOfImage + mi->ImageSize), is_embedded ? "\\" : get_symtype_str(mi), mi->ModuleName); } @@ -221,7 +221,8 @@ void info_win32_module(DWORD64 base) qsort(im.mi, im.num_used, sizeof(im.mi[0]), module_compare); - dbg_printf("Module\tAddress\t\t\tDebug info\tName (%d modules)\n", im.num_used); + dbg_printf("Module\tAddress\t\t\t%sDebug info\tName (%d modules)\n", + ADDRWIDTH == 16 ? "\t\t" : "", im.num_used); for (i = 0; i < im.num_used; i++) { @@ -370,10 +371,10 @@ static void info_window(HWND hWnd, int indent) if (!GetWindowTextA(hWnd, wndName, sizeof(wndName))) strcpy(wndName, "-- Empty --"); - dbg_printf("%*s%08lx%*s %-17.17s %08x %08lx %08x %.14s\n", + dbg_printf("%*s%08lx%*s %-17.17s %08x %0*lx %08x %.14s\n", indent, "", (DWORD_PTR)hWnd, 12 - indent, "", clsName, GetWindowLongW(hWnd, GWL_STYLE), - (ULONG_PTR)GetWindowLongPtrW(hWnd, GWLP_WNDPROC), + ADDRWIDTH, (ULONG_PTR)GetWindowLongPtrW(hWnd, GWLP_WNDPROC), GetWindowThreadProcessId(hWnd, NULL), wndName); if ((child = GetWindow(hWnd, GW_CHILD)) != 0) @@ -393,8 +394,9 @@ void info_win32_window(HWND hWnd, BOOL detailed) if (!detailed) { - dbg_printf("%-20.20s %-17.17s %-8.8s %-8.8s %-8.8s %s\n", - "Window handle", "Class Name", "Style", "WndProc", "Thread", "Text"); + dbg_printf("%-20.20s %-17.17s %-8.8s %-*.*s %-8.8s %s\n", + "Window handle", "Class Name", "Style", + ADDRWIDTH, ADDRWIDTH, "WndProc", "Thread", "Text"); info_window(hWnd, 0); return; } @@ -412,7 +414,7 @@ void info_win32_window(HWND hWnd, BOOL detailed) /* FIXME missing fields: hmemTaskQ, hrgnUpdate, dce, flags, pProp, scroll */ dbg_printf("next=%p child=%p parent=%p owner=%p class='%s'\n" "inst=%p active=%p idmenu=%08lx\n" - "style=0x%08x exstyle=0x%08x wndproc=0x%08lx text='%s'\n" + "style=0x%08x exstyle=0x%08x wndproc=%p text='%s'\n" "client=%d,%d-%d,%d window=%d,%d-%d,%d sysmenu=%p\n", GetWindow(hWnd, GW_HWNDNEXT), GetWindow(hWnd, GW_CHILD), @@ -424,7 +426,7 @@ void info_win32_window(HWND hWnd, BOOL detailed) (ULONG_PTR)GetWindowLongPtrW(hWnd, GWLP_ID), GetWindowLongW(hWnd, GWL_STYLE), GetWindowLongW(hWnd, GWL_EXSTYLE), - (ULONG_PTR)GetWindowLongPtrW(hWnd, GWLP_WNDPROC), + (void*)GetWindowLongPtrW(hWnd, GWLP_WNDPROC), wndName, clientRect.left, clientRect.top, clientRect.right, clientRect.bottom, windowRect.left, windowRect.top, windowRect.right, windowRect.bottom,

1 0

Eric Pouech : dbghelp: Don' t return error when validating a 64bit wide address on a 64bit platform.
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: 9ff52fb82cd764c48069edf51fb4f0ad5c23400a URL: http://source.winehq.org/git/wine.git/?a=commit;h=9ff52fb82cd764c48069edf51… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Dec 12 12:18:50 2009 +0100 dbghelp: Don't return error when validating a 64bit wide address on a 64bit platform. --- dlls/dbghelp/dbghelp.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/dbghelp/dbghelp.c b/dlls/dbghelp/dbghelp.c index 3f85f0d..ae26518 100644 --- a/dlls/dbghelp/dbghelp.c +++ b/dlls/dbghelp/dbghelp.c @@ -106,7 +106,7 @@ struct process* process_find_by_handle(HANDLE hProcess) */ BOOL validate_addr64(DWORD64 addr) { - if (addr >> 32) + if (sizeof(void*) == sizeof(int) && (addr >> 32)) { FIXME("Unsupported address %s\n", wine_dbgstr_longlong(addr)); SetLastError(ERROR_INVALID_PARAMETER);

1 0

Eric Pouech : winedbg: Enforce 64bit modules address on 64bit platforms.
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: 485280c64b9f361ecd31ebe43c93f731c69a84f7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=485280c64b9f361ecd31ebe43… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Dec 12 12:19:08 2009 +0100 winedbg: Enforce 64bit modules address on 64bit platforms. --- programs/winedbg/debugger.h | 2 +- programs/winedbg/types.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/programs/winedbg/debugger.h b/programs/winedbg/debugger.h index d0a0674..61325e3 100644 --- a/programs/winedbg/debugger.h +++ b/programs/winedbg/debugger.h @@ -94,7 +94,7 @@ enum dbg_internal_types struct dbg_type { unsigned long id; - DWORD module; + DWORD_PTR module; }; struct dbg_lvalue /* structure to hold left-values... */ diff --git a/programs/winedbg/types.c b/programs/winedbg/types.c index 50695aa..68bf12b 100644 --- a/programs/winedbg/types.c +++ b/programs/winedbg/types.c @@ -561,7 +561,7 @@ static BOOL CALLBACK print_types_cb(PSYMBOL_INFO sym, ULONG size, void* ctx) struct dbg_type type; type.module = sym->ModBase; type.id = sym->TypeIndex; - dbg_printf("Mod: %08x ID: %08lx\n", type.module, type.id); + dbg_printf("Mod: %08lx ID: %08lx\n", type.module, type.id); types_print_type(&type, TRUE); dbg_printf("\n"); return TRUE;

1 0

Eric Pouech : dbghelp: A couple of fixes for dwarf2 support on a 64bit platform.
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: b4da46ebe802bcc58e1dd53b374a938d556f2e8d URL: http://source.winehq.org/git/wine.git/?a=commit;h=b4da46ebe802bcc58e1dd53b3… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Dec 12 12:19:02 2009 +0100 dbghelp: A couple of fixes for dwarf2 support on a 64bit platform. --- dlls/dbghelp/dwarf.c | 21 ++++++++++++++++++--- 1 files changed, 18 insertions(+), 3 deletions(-) diff --git a/dlls/dbghelp/dwarf.c b/dlls/dbghelp/dwarf.c index bb7257e..4cd75f2 100644 --- a/dlls/dbghelp/dwarf.c +++ b/dlls/dbghelp/dwarf.c @@ -217,7 +217,7 @@ static unsigned char dwarf2_parse_byte(dwarf2_traverse_context_t* ctx) static unsigned short dwarf2_get_u2(const unsigned char* ptr) { - return *(const unsigned short*)ptr; + return *(const UINT16*)ptr; } static unsigned short dwarf2_parse_u2(dwarf2_traverse_context_t* ctx) @@ -229,7 +229,7 @@ static unsigned short dwarf2_parse_u2(dwarf2_traverse_context_t* ctx) static unsigned long dwarf2_get_u4(const unsigned char* ptr) { - return *(const unsigned long*)ptr; + return *(const UINT32*)ptr; } static unsigned long dwarf2_parse_u4(dwarf2_traverse_context_t* ctx) @@ -239,6 +239,11 @@ static unsigned long dwarf2_parse_u4(dwarf2_traverse_context_t* ctx) return uvalue; } +static DWORD64 dwarf2_get_u8(const unsigned char* ptr) +{ + return *(const UINT64*)ptr; +} + static unsigned long dwarf2_get_leb128_as_unsigned(const unsigned char* ptr, const unsigned char** end) { unsigned long ret = 0; @@ -309,6 +314,13 @@ static unsigned dwarf2_leb128_length(const dwarf2_traverse_context_t* ctx) return ret + 1; } +/****************************************************************** + * dwarf2_get_addr + * + * Returns an address. + * We assume that in all cases word size from Dwarf matches the size of + * addresses in platform where the exec is compiled. + */ static unsigned long dwarf2_get_addr(const unsigned char* ptr, unsigned word_size) { unsigned long ret; @@ -318,6 +330,9 @@ static unsigned long dwarf2_get_addr(const unsigned char* ptr, unsigned word_siz case 4: ret = dwarf2_get_u4(ptr); break; + case 8: + ret = dwarf2_get_u8(ptr); + break; default: FIXME("Unsupported Word Size %u\n", word_size); ret = 0; @@ -2033,7 +2048,7 @@ static BOOL dwarf2_parse_compilation_unit(const dwarf2_section_t* sections, cu_ctx.word_size = dwarf2_parse_byte(&cu_ctx); TRACE("Compilation Unit Header found at 0x%x:\n", - comp_unit_start - sections[section_debug].address); + (int)(comp_unit_start - sections[section_debug].address)); TRACE("- length: %lu\n", cu_length); TRACE("- version: %u\n", cu_version); TRACE("- abbrev_offset: %lu\n", cu_abbrev_offset);

1 0

Eric Pouech : dbghelp: Enforce using 64bit integers for module' s address and size.
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: c6eb8104dd31f1bbdd819cbfef6b9ffed9b88487 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c6eb8104dd31f1bbdd819cbfe… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Dec 12 12:18:56 2009 +0100 dbghelp: Enforce using 64bit integers for module's address and size. --- dlls/dbghelp/dbghelp_private.h | 4 ++-- dlls/dbghelp/module.c | 11 ++++++----- dlls/dbghelp/pe_module.c | 2 +- 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/dlls/dbghelp/dbghelp_private.h b/dlls/dbghelp/dbghelp_private.h index bbf2ef0..328c79b 100644 --- a/dlls/dbghelp/dbghelp_private.h +++ b/dlls/dbghelp/dbghelp_private.h @@ -473,7 +473,7 @@ extern BOOL module_get_debug(struct module_pair*); extern struct module* module_new(struct process* pcs, const WCHAR* name, enum module_type type, BOOL virtual, - unsigned long addr, unsigned long size, + DWORD64 addr, DWORD64 size, unsigned long stamp, unsigned long checksum); extern struct module* module_get_containee(const struct process* pcs, @@ -505,7 +505,7 @@ extern struct module* HANDLE hFile, DWORD base, DWORD size); extern struct module* pe_load_builtin_module(struct process* pcs, const WCHAR* name, - DWORD base, DWORD size); + DWORD64 base, DWORD64 size); extern BOOL pe_load_debug_info(const struct process* pcs, struct module* module); /* source.c */ diff --git a/dlls/dbghelp/module.c b/dlls/dbghelp/module.c index 1920dca..55eca44 100644 --- a/dlls/dbghelp/module.c +++ b/dlls/dbghelp/module.c @@ -122,7 +122,7 @@ static const char* get_module_type(enum module_type type, BOOL virtual) */ struct module* module_new(struct process* pcs, const WCHAR* name, enum module_type type, BOOL virtual, - unsigned long mod_addr, unsigned long size, + DWORD64 mod_addr, DWORD64 size, unsigned long stamp, unsigned long checksum) { struct module* module; @@ -134,8 +134,9 @@ struct module* module_new(struct process* pcs, const WCHAR* name, module->next = pcs->lmodules; pcs->lmodules = module; - TRACE("=> %s %08lx-%08lx %s\n", - get_module_type(type, virtual), mod_addr, mod_addr + size, + TRACE("=> %s %s-%s %s\n", + get_module_type(type, virtual), + wine_dbgstr_longlong(mod_addr), wine_dbgstr_longlong(mod_addr + size), debugstr_w(name)); pool_init(&module->pool, 65536); @@ -373,7 +374,7 @@ struct module* module_find_by_addr(const struct process* pcs, unsigned long addr * already loaded */ static BOOL module_is_container_loaded(const struct process* pcs, - const WCHAR* ImageName, DWORD base) + const WCHAR* ImageName, DWORD64 base) { size_t len; struct module* module; @@ -536,7 +537,7 @@ DWORD64 WINAPI SymLoadModuleExW(HANDLE hProcess, HANDLE hFile, PCWSTR wImageNam { if (!wImageName) return FALSE; module = module_new(pcs, wImageName, module_get_type_by_name(wImageName), - TRUE, (DWORD)BaseOfDll, SizeOfDll, 0, 0); + TRUE, BaseOfDll, SizeOfDll, 0, 0); if (!module) return FALSE; if (wModuleName) module_set_module(module, wModuleName); module->module.SymType = SymVirtual; diff --git a/dlls/dbghelp/pe_module.c b/dlls/dbghelp/pe_module.c index 664affc..03e9c86 100644 --- a/dlls/dbghelp/pe_module.c +++ b/dlls/dbghelp/pe_module.c @@ -448,7 +448,7 @@ BOOL pe_load_nt_header(HANDLE hProc, DWORD64 base, IMAGE_NT_HEADERS* nth) * */ struct module* pe_load_builtin_module(struct process* pcs, const WCHAR* name, - DWORD base, DWORD size) + DWORD64 base, DWORD64 size) { struct module* module = NULL;

1 0

Eric Pouech : dbghelp: Force 64bit module enumeration on all platforms.
by Alexandre Julliard Dec. 14, 2009

Dec. 14, 2009

Module: wine Branch: master Commit: 97d44dd39969adc741acd271350c7292c6344f53 URL: http://source.winehq.org/git/wine.git/?a=commit;h=97d44dd39969adc741acd2713… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sat Dec 12 12:19:23 2009 +0100 dbghelp: Force 64bit module enumeration on all platforms. --- programs/winedbg/debugger.h | 2 +- programs/winedbg/gdbproxy.c | 2 +- programs/winedbg/info.c | 1 - programs/winedbg/winedbg.c | 6 +++--- 4 files changed, 5 insertions(+), 6 deletions(-) diff --git a/programs/winedbg/debugger.h b/programs/winedbg/debugger.h index 42f9b24..30846eb 100644 --- a/programs/winedbg/debugger.h +++ b/programs/winedbg/debugger.h @@ -458,7 +458,7 @@ extern struct dbg_thread* dbg_get_thread(struct dbg_process* p, DWORD tid); extern void dbg_del_thread(struct dbg_thread* t); extern BOOL dbg_init(HANDLE hProc, const WCHAR* in, BOOL invade); extern BOOL dbg_load_module(HANDLE hProc, HANDLE hFile, const WCHAR* name, DWORD_PTR base, DWORD size); -extern BOOL dbg_get_debuggee_info(HANDLE hProcess, IMAGEHLP_MODULE* imh_mod); +extern BOOL dbg_get_debuggee_info(HANDLE hProcess, IMAGEHLP_MODULE64* imh_mod); extern void dbg_set_option(const char*, const char*); extern void dbg_start_interactive(HANDLE hFile); extern void dbg_init_console(void); diff --git a/programs/winedbg/gdbproxy.c b/programs/winedbg/gdbproxy.c index 4a63232..65b1cc3 100644 --- a/programs/winedbg/gdbproxy.c +++ b/programs/winedbg/gdbproxy.c @@ -2186,7 +2186,7 @@ static BOOL gdb_startup(struct gdb_context* gdbctx, DEBUG_EVENT* de, unsigned fl struct sockaddr_in s_addrs; unsigned int s_len = sizeof(s_addrs); struct pollfd pollfd; - IMAGEHLP_MODULE imh_mod; + IMAGEHLP_MODULE64 imh_mod; /* step 1: create socket for gdb connection request */ if ((sock = socket(AF_INET, SOCK_STREAM, 0)) == -1) diff --git a/programs/winedbg/info.c b/programs/winedbg/info.c index 87fbebd..1b7eeb3 100644 --- a/programs/winedbg/info.c +++ b/programs/winedbg/info.c @@ -146,7 +146,6 @@ static const char* get_symtype_str(const IMAGEHLP_MODULE64* mi) struct info_module { IMAGEHLP_MODULE64* mi; - DWORD_PTR _base; unsigned num_alloc; unsigned num_used; }; diff --git a/programs/winedbg/winedbg.c b/programs/winedbg/winedbg.c index 911cb6a..4aa0a3e 100644 --- a/programs/winedbg/winedbg.c +++ b/programs/winedbg/winedbg.c @@ -399,7 +399,7 @@ BOOL dbg_init(HANDLE hProc, const WCHAR* in, BOOL invade) struct mod_loader_info { HANDLE handle; - IMAGEHLP_MODULE* imh_mod; + IMAGEHLP_MODULE64* imh_mod; }; static BOOL CALLBACK mod_loader_cb(PCSTR mod_name, DWORD64 base, PVOID ctx) @@ -408,13 +408,13 @@ static BOOL CALLBACK mod_loader_cb(PCSTR mod_name, DWORD64 base, PVOID ctx) if (!strcmp(mod_name, "<wine-loader>")) { - if (SymGetModuleInfo(mli->handle, base, mli->imh_mod)) + if (SymGetModuleInfo64(mli->handle, base, mli->imh_mod)) return FALSE; /* stop enum */ } return TRUE; } -BOOL dbg_get_debuggee_info(HANDLE hProcess, IMAGEHLP_MODULE* imh_mod) +BOOL dbg_get_debuggee_info(HANDLE hProcess, IMAGEHLP_MODULE64* imh_mod) { struct mod_loader_info mli; DWORD opt;

1 0