[ros-diffs] [reactos] 105/179: [WINESYNC] dbghelp: Use search_unix_path to load seach macho files.

[winesync]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=8194b5dee6d4d1341803e…

commit 8194b5dee6d4d1341803e848809e574166a1fb46
Author:     winesync <ros-dev(a)reactos.org>
AuthorDate: Fri Sep 11 17:10:41 2020 +0200
Commit:     Jérôme Gardou <jerome.gardou(a)reactos.org>
CommitDate: Wed Sep 16 10:35:47 2020 +0200

    [WINESYNC] dbghelp: Use search_unix_path to load seach macho files.
    
    Signed-off-by: Jacek Caban <jacek(a)codeweavers.com>
    Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>
    
    wine commit id c3bb866bfc53b8c3b7b7355cb8311325a40a0a6b by Jacek Caban
<jacek(a)codeweavers.com>
---
 dll/win32/dbghelp/macho_module.c | 63 +++++++++++-----------------------------
 sdk/tools/winesync/dbghelp.cfg   |  2 +-
 2 files changed, 18 insertions(+), 47 deletions(-)

diff --git a/dll/win32/dbghelp/macho_module.c b/dll/win32/dbghelp/macho_module.c
index 4bc0a3e903c..038319ce35c 100644
--- a/dll/win32/dbghelp/macho_module.c
+++ b/dll/win32/dbghelp/macho_module.c
@@ -1523,49 +1523,17 @@ leave:
     return ret;
 }
 
-/******************************************************************
- *              macho_load_file_from_path
- * Tries to load a Mach-O file from a set of paths (separated by ':')
- */
-static BOOL macho_load_file_from_path(struct process* pcs,
-                                      const WCHAR* filename,
-                                      unsigned long load_addr,
-                                      const char* path,
-                                      struct macho_info* macho_info)
+struct macho_load_params
 {
-    BOOL                ret = FALSE;
-    WCHAR               *s, *t, *fn;
-    WCHAR*              pathW = NULL;
-    unsigned            len;
-
-    TRACE("(%p/%p, %s, 0x%08lx, %s, %p)\n", pcs, pcs->handle,
debugstr_w(filename), load_addr,
-            debugstr_a(path), macho_info);
-
-    if (!path) return FALSE;
-
-    len = MultiByteToWideChar(CP_UNIXCP, 0, path, -1, NULL, 0);
-    pathW = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR));
-    if (!pathW) return FALSE;
-    MultiByteToWideChar(CP_UNIXCP, 0, path, -1, pathW, len);
-
-    for (s = pathW; s && *s; s = (t) ? (t+1) : NULL)
-    {
-        t = strchrW(s, ':');
-        if (t) *t = '\0';
-        fn = HeapAlloc(GetProcessHeap(), 0, (lstrlenW(filename) + 1 + lstrlenW(s) + 1) *
sizeof(WCHAR));
-        if (!fn) break;
-        strcpyW(fn, s);
-        strcatW(fn, S_SlashW);
-        strcatW(fn, filename);
-        ret = macho_load_file(pcs, fn, load_addr, macho_info);
-        HeapFree(GetProcessHeap(), 0, fn);
-        if (ret) break;
-        s = (t) ? (t+1) : NULL;
-    }
+    struct process *process;
+    ULONG_PTR load_addr;
+    struct macho_info *macho_info;
+};
 
-    TRACE(" => %d\n", ret);
-    HeapFree(GetProcessHeap(), 0, pathW);
-    return ret;
+static BOOL macho_load_file_cb(void *param, HANDLE handle, const WCHAR *filename)
+{
+    struct macho_load_params *macho_load = param;
+    return macho_load_file(macho_load->process, filename, macho_load->load_addr,
macho_load->macho_info);
 }
 
 /******************************************************************
@@ -1619,6 +1587,7 @@ static BOOL macho_search_and_load_file(struct process* pcs, const
WCHAR* filenam
     struct module*      module;
     static const WCHAR  S_libstdcPPW[] =
{'l','i','b','s','t','d','c','+','+','\0'};
     const WCHAR*        p;
+    struct macho_load_params load_params;
 
     TRACE("(%p/%p, %s, 0x%08lx, %p)\n", pcs, pcs->handle,
debugstr_w(filename), load_addr,
             macho_info);
@@ -1633,17 +1602,19 @@ static BOOL macho_search_and_load_file(struct process* pcs, const
WCHAR* filenam
 
     if (strstrW(filename, S_libstdcPPW)) return FALSE; /* We know we can't do it */
 
+    load_params.process    = pcs;
+    load_params.load_addr  = load_addr;
+    load_params.macho_info = macho_info;
+
     /* If has no directories, try PATH first. */
     p = file_name(filename);
     if (p == filename)
     {
-        ret = macho_load_file_from_path(pcs, filename, load_addr,
-                                      getenv("PATH"), macho_info);
+        ret = search_unix_path(filename, getenv("PATH"), macho_load_file_cb,
&load_params);
     }
     /* Try DYLD_LIBRARY_PATH, with just the filename (no directories). */
     if (!ret)
-        ret = macho_load_file_from_path(pcs, p, load_addr,
-                                      getenv("DYLD_LIBRARY_PATH"),
macho_info);
+        ret = search_unix_path(p, getenv("DYLD_LIBRARY_PATH"),
macho_load_file_cb, &load_params);
 
     /* Try the path as given. */
     if (!ret)
@@ -1654,7 +1625,7 @@ static BOOL macho_search_and_load_file(struct process* pcs, const
WCHAR* filenam
         const char* fallback = getenv("DYLD_FALLBACK_LIBRARY_PATH");
         if (!fallback)
             fallback = "/usr/local/lib:/lib:/usr/lib";
-        ret = macho_load_file_from_path(pcs, p, load_addr, fallback, macho_info);
+        ret = search_unix_path(p, fallback, macho_load_file_cb, &load_params);
     }
     if (!ret && p == filename)
         ret = macho_load_file_from_dll_path(pcs, filename, load_addr, macho_info);
diff --git a/sdk/tools/winesync/dbghelp.cfg b/sdk/tools/winesync/dbghelp.cfg
index df209bd3ca0..0a7da2e8203 100644
--- a/sdk/tools/winesync/dbghelp.cfg
+++ b/sdk/tools/winesync/dbghelp.cfg
@@ -4,4 +4,4 @@ files:
   include/dbghelp.h: sdk/include/psdk/dbghelp.h
   include/wine/mscvpdb.h: sdk/include/reactos/wine/mscvpdb.h
 tags:
-  wine: ca49552f646bf30b492da4668745f9f65b9de18d
+  wine: c3bb866bfc53b8c3b7b7355cb8311325a40a0a6b