[ros-diffs] [reactos] 21/179: [WINESYNC] dbghelp: Read the Mach-O segment command corresponding to the target's architecture.

[winesync]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=75e77444c6be66dacd390…

commit 75e77444c6be66dacd390e163e6cf274c3edd5f0
Author:     winesync <ros-dev(a)reactos.org>
AuthorDate: Fri Sep 11 13:14:05 2020 +0200
Commit:     Jérôme Gardou <jerome.gardou(a)reactos.org>
CommitDate: Wed Sep 16 10:35:33 2020 +0200

    [WINESYNC] dbghelp: Read the Mach-O segment command corresponding to the target's
architecture.
    
    Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com>
    Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>
    
    wine commit id 0fc3c33637d4c8eff3f4373d374b3feb402bce4a by Zebediah Figura
<zfigura(a)codeweavers.com>
---
 dll/win32/dbghelp/macho_module.c | 68 +++++++++++++++++++++++++++++-----------
 sdk/tools/winesync/dbghelp.cfg   |  2 +-
 2 files changed, 51 insertions(+), 19 deletions(-)

diff --git a/dll/win32/dbghelp/macho_module.c b/dll/win32/dbghelp/macho_module.c
index 50fcca4d41f..52ea739b13a 100644
--- a/dll/win32/dbghelp/macho_module.c
+++ b/dll/win32/dbghelp/macho_module.c
@@ -89,10 +89,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(dbghelp_macho);
 
 
 #ifdef _WIN64
-typedef struct segment_command_64   macho_segment_command;
 typedef struct nlist_64             macho_nlist;
 #else
-typedef struct segment_command      macho_segment_command;
 typedef struct nlist                macho_nlist;
 #endif
 
@@ -506,12 +504,26 @@ static int macho_enum_load_commands(struct image_file_map *ifm,
unsigned cmd,
  */
 static int macho_count_sections(struct image_file_map* ifm, const struct load_command*
lc, void* user)
 {
-    const macho_segment_command* sc = (const macho_segment_command*)lc;
+    char segname[16];
+    uint32_t nsects;
+
+    if (ifm->addr_size == 32)
+    {
+        const struct segment_command *sc = (const struct segment_command *)lc;
+        memcpy(segname, sc->segname, sizeof(segname));
+        nsects = sc->nsects;
+    }
+    else
+    {
+        const struct segment_command_64 *sc = (const struct segment_command_64 *)lc;
+        memcpy(segname, sc->segname, sizeof(segname));
+        nsects = sc->nsects;
+    }
 
     TRACE("(%p/%d, %p, %p) segment %s\n", ifm, ifm->u.macho.fd, lc, user,
-        debugstr_an(sc->segname, sizeof(sc->segname)));
+        debugstr_an(segname, sizeof(segname)));
 
-    ifm->u.macho.num_sections += sc->nsects;
+    ifm->u.macho.num_sections += nsects;
     return 0;
 }
 
@@ -525,44 +537,64 @@ static int macho_count_sections(struct image_file_map* ifm, const
struct load_co
 static int macho_load_section_info(struct image_file_map* ifm, const struct load_command*
lc, void* user)
 {
     struct macho_file_map*          fmap = &ifm->u.macho;
-    const macho_segment_command*    sc = (const macho_segment_command*)lc;
     struct section_info*            info = user;
     BOOL                            ignore;
     const macho_section*            section;
     int                             i;
     unsigned long                   tmp, page_mask = sysconf( _SC_PAGESIZE ) - 1;
+    uint64_t vmaddr, vmsize;
+    char segname[16];
+    uint32_t nsects;
+
+    if (ifm->addr_size == 32)
+    {
+        const struct segment_command *sc = (const struct segment_command *)lc;
+        vmaddr = sc->vmaddr;
+        vmsize = sc->vmsize;
+        memcpy(segname, sc->segname, sizeof(segname));
+        nsects = sc->nsects;
+        section = (const macho_section*)(sc + 1);
+    }
+    else
+    {
+        const struct segment_command_64 *sc = (const struct segment_command_64 *)lc;
+        vmaddr = sc->vmaddr;
+        vmsize = sc->vmsize;
+        memcpy(segname, sc->segname, sizeof(segname));
+        nsects = sc->nsects;
+        section = (const macho_section*)(sc + 1);
+    }
 
     TRACE("(%p/%d, %p, %p) before: 0x%08lx - 0x%08lx\n", fmap, fmap->fd, lc,
user,
             (unsigned long)fmap->segs_start, (unsigned long)fmap->segs_size);
-    TRACE("Segment command vm: 0x%08lx - 0x%08lx\n", (unsigned
long)sc->vmaddr,
-            (unsigned long)(sc->vmaddr + sc->vmsize));
+    TRACE("Segment command vm: 0x%08lx - 0x%08lx\n", (unsigned long)vmaddr,
+            (unsigned long)(vmaddr + vmsize));
 
     /* Images in the dyld shared cache have their segments mapped non-contiguously.
        We don't know how to properly locate any of the segments other than __TEXT,
        so ignore them. */
-    ignore = (info->split_segs && strcmp(sc->segname, SEG_TEXT));
+    ignore = (info->split_segs && strcmp(segname, SEG_TEXT));
 
-    if (!strncmp(sc->segname, "WINE_", 5))
-        TRACE("Ignoring special Wine segment %s\n", debugstr_an(sc->segname,
sizeof(sc->segname)));
-    else if (!strncmp(sc->segname, "__PAGEZERO", 10))
+    if (!strncmp(segname, "WINE_", 5))
+        TRACE("Ignoring special Wine segment %s\n", debugstr_an(segname,
sizeof(segname)));
+    else if (!strncmp(segname, "__PAGEZERO", 10))
         TRACE("Ignoring __PAGEZERO segment\n");
     else if (ignore)
-        TRACE("Ignoring %s segment because image has split segments\n",
sc->segname);
+        TRACE("Ignoring %s segment because image has split segments\n",
segname);
     else
     {
         /* If this segment starts before previously-known earliest, record new earliest.
*/
-        if (sc->vmaddr < fmap->segs_start)
-            fmap->segs_start = sc->vmaddr;
+        if (vmaddr < fmap->segs_start)
+            fmap->segs_start = vmaddr;
 
         /* If this segment extends beyond previously-known furthest, record new furthest.
*/
-        tmp = (sc->vmaddr + sc->vmsize + page_mask) & ~page_mask;
+        tmp = (vmaddr + vmsize + page_mask) & ~page_mask;
         if (fmap->segs_size < tmp) fmap->segs_size = tmp;
 
         TRACE("after: 0x%08lx - 0x%08lx\n", (unsigned long)fmap->segs_start,
(unsigned long)fmap->segs_size);
     }
 
-    section = (const macho_section*)(sc + 1);
-    for (i = 0; i < sc->nsects; i++)
+    for (i = 0; i < nsects; i++)
     {
         fmap->sect[info->section_index].section = &section[i];
         fmap->sect[info->section_index].mapped = IMAGE_NO_MAP;
diff --git a/sdk/tools/winesync/dbghelp.cfg b/sdk/tools/winesync/dbghelp.cfg
index 47ba99d45ee..7b8de6b1dbd 100644
--- a/sdk/tools/winesync/dbghelp.cfg
+++ b/sdk/tools/winesync/dbghelp.cfg
@@ -3,4 +3,4 @@ directories:
 files:
   include/dbghelp.h: sdk/include/psdk/dbghelp.h
 tags:
-  wine: bf7681cb2f3a73239170b4f0ebc8f6b38bc027ec
+  wine: 0fc3c33637d4c8eff3f4373d374b3feb402bce4a