[PATCH 1/5] winedbg: Use target pointer size in indirect string access.

Rémi Bernon rbernon at codeweavers.com
Sat Apr 4 04:30:13 CDT 2020


We read it into a void* so we also need to zero initialize it in case
the target pointer size is shorter than ours.

Signed-off-by: Rémi Bernon <rbernon at codeweavers.com>
---
 programs/winedbg/memory.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/programs/winedbg/memory.c b/programs/winedbg/memory.c
index 053ace94ac81..c9b97686aab0 100644
--- a/programs/winedbg/memory.c
+++ b/programs/winedbg/memory.c
@@ -268,12 +268,12 @@ BOOL memory_get_string(struct dbg_process* pcs, void* addr, BOOL in_debuggee,
 
 BOOL memory_get_string_indirect(struct dbg_process* pcs, void* addr, BOOL unicode, WCHAR* buffer, int size)
 {
-    void*       ad;
+    void*       ad = 0;
     SIZE_T	sz;
 
     buffer[0] = 0;
     if (addr &&
-        pcs->process_io->read(pcs->handle, addr, &ad, sizeof(ad), &sz) && sz == sizeof(ad) && ad)
+        pcs->process_io->read(pcs->handle, addr, &ad, pcs->be_cpu->pointer_size, &sz) && sz == pcs->be_cpu->pointer_size && ad)
     {
         LPSTR buff;
         BOOL ret;
-- 
2.26.0




More information about the wine-devel mailing list