From ed0ccdd6e36671267c622c6442267bbbe1caf7e4 Mon Sep 17 00:00:00 2001
From: bicarus <202771338+bicarus-dev@users.noreply.github.com>
Date: Sat, 3 Oct 2026 14:14:22 -0700
Subject: [PATCH] launcher: better DLL logging and remove ucrt hack (#919)

## Link to GitHub Issue or related Pull Request, if one exists
n/a

## Description of change
Remove workaround implemented in #264 that added an ugly hack to work
around DLL hooks that rely on spice using MSVCRT (failure to obtain
argc/argv).

Update the logic that prints info about DLL overrides so that it tries
to print out as much info as possible, instead of bailing out early if
something is missing.

## Testing
---
 src/spice2x/launcher/launcher.cpp |  74 +++++-----------
 src/spice2x/util/libutils.cpp     | 143 +++++++++++++++++++++---------
 src/spice2x/util/libutils.h       |   2 +-
 3 files changed, 122 insertions(+), 97 deletions(-)

diff --git a/src/spice2x/launcher/launcher.cpp b/src/spice2x/launcher/launcher.cpp
index f600d15a..37210648 100644
--- a/src/spice2x/launcher/launcher.cpp
+++ b/src/spice2x/launcher/launcher.cpp
@@ -172,8 +172,6 @@ static bool check_dll(const std::string &model) {
     }
 }
 
-void update_msvcrt_args(int argc, char *argv[]);
-
 void dump_button_bindings(std::vector<Button> *buttons);
 void dump_analog_bindings();
 
@@ -2528,7 +2526,23 @@ int main_implementation(int argc, char *argv[]) {
     // log some DLLs found in path (purely for troubleshooting purposes to detect
     // dxvk, ForceD3D9On12, ifs_layeredfs, etc)
     libutils::warn_if_dll_exists("d3d8.dll");
-    libutils::warn_if_dll_exists("d3d9.dll");
+    if (libutils::warn_if_dll_exists("d3d9.dll")) {
+
+#if SPICE64
+        // dx9 subscreen games
+        if (avs::game::is_model({"LDJ", "KFC", "M39", "M32"})) {
+            log_warning(
+                "launcher",
+                "custom d3d9.dll detected - may cause graphics and touch-emulation issues");
+            deferredlogs::defer_error_messages({
+                "found custom d3d9.dll",
+                "    custom d3d9.dll wrappers may cause subscreen and touch-emulation issues",
+                "    if you encounter problems, close the game, remove the custom d3d9.dll and retry"
+            });
+        }
+#endif
+    }
+
     libutils::warn_if_dll_exists("d3d10core.dll");
     libutils::warn_if_dll_exists("d3d11.dll");
     libutils::warn_if_dll_exists("d3d12.dll");
@@ -2745,7 +2759,11 @@ int main_implementation(int argc, char *argv[]) {
         networkhook_init();
     }
 
-    update_msvcrt_args(argc, argv);
+#if defined(_UCRT) || (defined(_MSC_VER) && _MSC_VER >= 1900)
+    log_info("launcher", "C runtime: UCRT");
+#else
+    log_info("launcher", "C runtime: MSVCRT");
+#endif
 
     // load hooks
     for (auto &hook : game_hooks) {
@@ -2986,54 +3004,6 @@ int main_implementation(int argc, char *argv[]) {
     return 0;
 }
 
-// https://github.com/spice2x/spice2x.github.io/issues/264
-// huge ugly hack to work around things that broke when MinGW switched from msvcrt to ucrt
-// this is done to ensure that any DLL hooks that rely on msvcrt continue to work
-void update_msvcrt_args(int argc, char *argv[]) {
-#if defined(_UCRT)
-    auto msvc = LoadLibraryA("msvcrt.dll");
-    if (!msvc) {
-        log_warning("launcher", "failed to load msvcrt.dll");
-        return;
-    }
-
-    // get __argc
-    PINT32 argc_addr = (PINT32)GetProcAddress(msvc, "__argc");
-    if (!argc_addr) {
-        log_warning("launcher", "failed to find msvcrt!__argc");
-        return;
-    }
-    try {
-        if (*argc_addr == argc) {
-            log_warning("launcher", "msvcrt!__argc is already set");
-            return;
-        }
-    } catch (const std::exception &e) {
-        log_warning("launcher", "exception while reading msvcrt!_argc: {}", e.what());
-    }
-
-    // get __argv
-    PCHAR **argv_addr = (PCHAR **)GetProcAddress(msvc, "__argv");
-    if (!argv_addr) {
-        log_warning("launcher", "failed to find msvcrt!__argv");
-        return;
-    }
-
-    // update them
-    try {
-        log_info("launcher", "msvcrt!__argc value before: {}", *argc_addr);
-        *argc_addr = argc;
-        log_info("launcher", "msvcrt!__argc value after: {}", *argc_addr);
-        *argv_addr = argv;
-    } catch (const std::exception &e) {
-        log_warning("launcher", "exception while messing with msvcrt!_argc and _argv: {}", e.what());
-    }
-
-#else
-    log_misc("launcher", "not UCRT, skipping msvcrt!_argc / _argv hacks");
-#endif
-}
-
 void dump_button_bindings(std::vector<Button> *buttons) {
     if (!buttons) {
         return;
diff --git a/src/spice2x/util/libutils.cpp b/src/spice2x/util/libutils.cpp
index 0470672a..79b67e25 100644
--- a/src/spice2x/util/libutils.cpp
+++ b/src/spice2x/util/libutils.cpp
@@ -369,61 +369,106 @@ void libutils::check_duplicate_dlls() {
     }
 }
 
-void libutils::warn_if_dll_exists(const std::string &file_name) {
-    if (fileutils::file_exists(MODULE_PATH / file_name)) {
-        log_info("libutils", "found user-supplied {} in modules directory", file_name);
-        libutils::print_dll_info(MODULE_PATH / file_name);
-        return;
-    }
+bool libutils::warn_if_dll_exists(const std::string &file_name) {
     const auto &spice_bin_path = libutils::module_file_name(nullptr).parent_path();
+    bool found = false;
     if (fileutils::file_exists(spice_bin_path / file_name)) {
         log_info("libutils", "found user-supplied {} next to spice executable path", file_name);
         libutils::print_dll_info(spice_bin_path / file_name);
-        return;
+        found = true;
     }
+    if (MODULE_PATH != spice_bin_path && fileutils::file_exists(MODULE_PATH / file_name)) {
+        log_info("libutils", "found user-supplied {} in modules directory", file_name);
+        libutils::print_dll_info(MODULE_PATH / file_name);
+        found = true;
+    }
+    return found;
 }
 
 void libutils::print_dll_info(std::filesystem::path filename) {
-    DWORD handle;
+    std::string company_name;
+    std::string product_name;
+    std::string version_str;
+    auto print_summary = [&]() {
+        log_info(
+            "libutils",
+            "DLL info for {}: CompanyName = {}, ProductName = {}, Version = {}, Path = {}",
+            filename.filename(),
+            company_name.empty() ? "?" : company_name,
+            product_name.empty() ? "?" : product_name,
+            version_str.empty() ? "?" : version_str,
+            filename);
+    };
+
+    DWORD handle = 0;
     const auto size = GetFileVersionInfoSizeW(filename.wstring().c_str(), &handle);
     if (size == 0) {
-        log_debug(
+        const auto error = GetLastError();
+        const auto error_message = get_last_error_string();
+        log_misc(
             "libutils",
-            "GetFileVersionInfoSizeA failed for {}: {}",
-            filename.filename(),
-            get_last_error_string());
+            "GetFileVersionInfoSizeW failed for {}: {} (Win32 error {}); "
+            "DLL may have no version resource",
+            filename,
+            error_message,
+            error);
+        print_summary();
         return;
     }
 
     auto data = util::make_unique_plain<VOID>(size);
     if (!GetFileVersionInfoW(filename.wstring().c_str(), handle, size, data.get())) {
-        log_debug(
+        const auto error = GetLastError();
+        const auto error_message = get_last_error_string();
+        log_misc(
             "libutils",
-            "GetFileVersionInfoA failed for {}: {}",
-            filename.filename(),
-            get_last_error_string());
+            "GetFileVersionInfoW failed for {}: {} (Win32 error {})",
+            filename,
+            error_message,
+            error);
+        print_summary();
         return;
     }
 
+    VS_FIXEDFILEINFO *fixed_info = nullptr;
+    UINT fixed_size = 0;
+    if (VerQueryValueW(data.get(), L"\\", (LPVOID*)&fixed_info, &fixed_size) &&
+        fixed_info != nullptr && fixed_size >= sizeof(*fixed_info) &&
+        fixed_info->dwSignature == 0xFEEF04BD) {
+        version_str = fmt::format(
+            "{}.{}.{}.{}",
+            HIWORD(fixed_info->dwFileVersionMS),
+            LOWORD(fixed_info->dwFileVersionMS),
+            HIWORD(fixed_info->dwFileVersionLS),
+            LOWORD(fixed_info->dwFileVersionLS));
+        log_info("libutils", "Fixed file version for {}: {}", filename.filename(), version_str);
+    } else {
+        log_misc("libutils", "Missing or invalid fixed version info for {}", filename);
+    }
+
     struct LANGANDCODEPAGE {
         WORD wLanguage;
         WORD wCodePage;
     } *lpTranslate = nullptr;
 
     UINT cbTranslate = 0;
-    if (!VerQueryValueA(data.get(), "\\VarFileInfo\\Translation", (LPVOID*)&lpTranslate, &cbTranslate)) {
-        log_debug(
+    if (!VerQueryValueW(
+            data.get(), L"\\VarFileInfo\\Translation", (LPVOID*)&lpTranslate, &cbTranslate)) {
+        log_misc(
             "libutils",
-            "VerQueryValueA failed for {}: {}",
-            filename.filename(),
-            get_last_error_string());
+            "VerQueryValueW(\\VarFileInfo\\Translation) failed for {}",
+            filename);
+        print_summary();
         return;
     }
-    if (cbTranslate == 0 || lpTranslate == nullptr) {
-        log_debug(
+    if (cbTranslate < sizeof(*lpTranslate) ||
+        cbTranslate % sizeof(*lpTranslate) != 0 || lpTranslate == nullptr) {
+        log_misc(
             "libutils",
-            "VerQueryValueA returned invalid results for {}",
-            filename.filename());
+            "Invalid version translation table for {} ({} bytes)",
+            filename,
+            cbTranslate);
+        print_summary();
         return;
     }
 
@@ -442,36 +487,46 @@ void libutils::print_dll_info(std::filesystem::path filename) {
     }
 
     // StringFileInfo helper
-    auto query_string = [&](const char* key) -> std::string {
+    auto query_string = [&](const char* key, WORD language, WORD page) -> std::string {
         std::string subBlock = fmt::format(
             "\\StringFileInfo\\{:04x}{:04x}\\{}",
-            lang, codepage, key
+            language, page, key
         );
 
-        char* value = nullptr;
+        wchar_t *value = nullptr;
         UINT size_out = 0;
 
-        if (VerQueryValueA(data.get(), subBlock.c_str(), (LPVOID*)&value, &size_out) && value) {
-            return value;
+        const auto wide_sub_block = s2ws(subBlock);
+        if (VerQueryValueW(data.get(), wide_sub_block.c_str(), (LPVOID*)&value, &size_out) &&
+            value != nullptr && size_out > 0 && value[size_out - 1] == L'\0') {
+            return fmt::detail::to_utf8<wchar_t>(
+                std::wstring_view(value, size_out - 1),
+                fmt::detail::to_utf8_error_policy::replace).str();
         }
-        log_debug(
+        log_misc(
             "libutils",
-            "VerQueryValueA({}) failed for {}: {}",
+            "Missing or invalid version string {} for {}",
             subBlock,
-            filename.filename(),
-            get_last_error_string());
+            filename);
         return "";
     };
 
-    const auto company_name = query_string("CompanyName");
-    const auto product_name = query_string("ProductName");
-    const auto version_str = query_string("FileVersion");
+    auto query_translations = [&](const char *key) {
+        auto value = query_string(key, lang, codepage);
+        for (UINT i = 0; value.empty() && i < cbTranslate / sizeof(*lpTranslate); i++) {
+            const auto &translation = lpTranslate[i];
+            if (translation.wLanguage != lang || translation.wCodePage != codepage) {
+                value = query_string(key, translation.wLanguage, translation.wCodePage);
+            }
+        }
+        return value;
+    };
 
-    log_info(
-        "libutils",
-        "DLL info for {}: CompanyName = {}, ProductName = {}, Version = {}",
-        filename.filename(),
-        company_name.empty() ? "?" : company_name,
-        product_name.empty() ? "?" : product_name,
-        version_str.empty() ? "?" : version_str);
+    company_name = query_translations("CompanyName");
+    product_name = query_translations("ProductName");
+    const auto string_version = query_translations("FileVersion");
+    if (!string_version.empty()) {
+        version_str = string_version;
+    }
+    print_summary();
 }
diff --git a/src/spice2x/util/libutils.h b/src/spice2x/util/libutils.h
index 0140756b..97da8c16 100644
--- a/src/spice2x/util/libutils.h
+++ b/src/spice2x/util/libutils.h
@@ -25,7 +25,7 @@ namespace libutils {
     }
 
     void check_duplicate_dlls();
-    void warn_if_dll_exists(const std::string &file_name);
+    bool warn_if_dll_exists(const std::string &file_name);
 
     // get module handle helpers
     HMODULE get_module(const char *module_name);