[PATCH v2 0/1] MR5449: Draft: regsvr32: Implement EXE registration.

[Aida Jonikienė]

The Sims 2 installer calls regsvr32 with the /regserver flag (which
isn't implemented on Wine) so that command fails.

--
  v2: regsvr32: Implement EXE registration.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5449

[PATCH v2 1/1] regsvr32: Implement EXE registration.

[Aida Jonikienė]

From: Aida Jonikienė <aidas957(a)gmail.com>

The Sims 2 installer calls regsvr32 with the /regserver flag (which
isn't implemented on Wine) so that command fails.
---
 programs/regsvr32/regsvr32.c | 96 ++++++++++++++++++++++++++++++++++++
 1 file changed, 96 insertions(+)

diff --git a/programs/regsvr32/regsvr32.c b/programs/regsvr32/regsvr32.c
index 18c664f436a..2d8f5fcc404 100644
--- a/programs/regsvr32/regsvr32.c
+++ b/programs/regsvr32/regsvr32.c
@@ -240,6 +240,76 @@ static int UnregisterDll(const WCHAR* strDll, BOOL firstDll)
     return 0;
 }
 
+static int RegisterExe(const WCHAR* strExe)
+{
+    WCHAR app[MAX_PATH];
+    WCHAR registerW[] = L" /regserver";
+    WCHAR *cmdline;
+    STARTUPINFOW si = {0};
+    PROCESS_INFORMATION pi;
+
+    wcscat(app, strExe);
+
+    TRACE("registering %s\n", debugstr_w(strExe));
+
+    cmdline = HeapAlloc(GetProcessHeap(), 0,
+                        (wcslen(app)+wcslen(registerW)+1)*sizeof(WCHAR));
+
+    wcscpy(cmdline, app);
+    wcscat(cmdline, registerW);
+
+    si.cb = sizeof(si);
+
+    if (CreateProcessW(app, cmdline, NULL, NULL, !Silent, 0, NULL, NULL, &si, &pi))
+    {
+        WaitForSingleObject(pi.hProcess, INFINITE);
+        output_write(FALSE, STRING_REGISTER_SUCCESSFUL, strExe);
+        HeapFree(GetProcessHeap(), 0, cmdline);
+        return 0;
+    }
+    else
+    {
+        HeapFree(GetProcessHeap(), 0, cmdline);
+        output_write(FALSE, STRING_REGISTER_FAILED, strExe);
+        return DLLSERVER_FAILED; /* FIXME: Is this the right value to return? */
+    }
+}
+
+static int UnregisterExe(const WCHAR* strExe)
+{
+    WCHAR app[MAX_PATH];
+    WCHAR unregisterW[] = L" /unregserver";
+    WCHAR *cmdline;
+    STARTUPINFOW si = {0};
+    PROCESS_INFORMATION pi;
+
+    wcscat(app, strExe);
+
+    TRACE("unregistering %s\n", debugstr_w(strExe));
+
+    cmdline = HeapAlloc(GetProcessHeap(), 0,
+                        (wcslen(app)+wcslen(unregisterW)+1)*sizeof(WCHAR));
+
+    wcscpy(cmdline, app);
+    wcscat(cmdline, unregisterW);
+
+    si.cb = sizeof(si);
+
+    if (CreateProcessW(app, cmdline, NULL, NULL, !Silent, 0, NULL, NULL, &si, &pi))
+    {
+        WaitForSingleObject(pi.hProcess, INFINITE);
+        output_write(FALSE, STRING_UNREGISTER_SUCCESSFUL, strExe);
+        HeapFree(GetProcessHeap(), 0, cmdline);
+        return 0;
+    }
+    else
+    {
+        HeapFree(GetProcessHeap(), 0, cmdline);
+        output_write(FALSE, STRING_UNREGISTER_FAILED, strExe);
+        return DLLSERVER_FAILED; /* FIXME: Is this the right value to return? */
+    }
+}
+
 static int InstallDll(BOOL install, const WCHAR *strDll, const WCHAR *command_line, BOOL firstDll)
 {
     HRESULT hr;
@@ -301,11 +371,19 @@ int __cdecl wmain(int argc, WCHAR* argv[])
     BOOL            CallInstall = FALSE;
     BOOL            Unregister = FALSE;
     BOOL            DllFound = FALSE;
+    BOOL            ExeRegister = FALSE;
+    BOOL            ExeUnregister = FALSE;
     WCHAR*          wsCommandLine = NULL;
     WCHAR           EmptyLine[] = L"";
 
     OleInitialize(NULL);
 
+    TRACE("Arguments:\n");
+    for (i = 1; i < argc; i++)
+    {
+        TRACE("%s\n", wine_dbgstr_w(argv[i]));
+    }
+
     /* We mirror the Microsoft version by processing all of the flags before
      * the files (e.g. regsvr32 file1 /s file2 is silent even for file1).
      *
@@ -321,6 +399,19 @@ int __cdecl wmain(int argc, WCHAR* argv[])
             if (!argv[i][1])
                 return INVALID_ARG;
 
+            if (!wcscmp(wcslwr(argv[i] + 1), L"regserver"))
+            {
+               ExeRegister = TRUE;
+               argv[i] = NULL;
+               continue;
+            }
+            else if (!wcscmp(wcslwr(argv[i] + 1), L"unregserver"))
+            {
+               ExeUnregister = TRUE;
+               argv[i] = NULL;
+               continue;
+            }
+
             if (argv[i][2] && argv[i][2] != ':')
                 continue;
 
@@ -371,6 +462,11 @@ int __cdecl wmain(int argc, WCHAR* argv[])
                 continue;
             }
 
+            if (ExeRegister)
+                res = RegisterExe(DllName);
+            else if (ExeUnregister)
+                res = UnregisterExe(DllName);
+
             if (!CallInstall || CallRegister)
             {
                 if(Unregister)
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/5449

[PATCH v2 0/1] MR5449: regsvr32: Implement EXE registration.

[Aida Jonikienė]

On Mon Apr 29 18:13:09 2024 +0000, Aida Jonikienė wrote:
> changed this line in [version 2 of the diff](/wine/wine/-/merge_requests/5449/diffs?diff_id=111678&start_sha=11b11c0dd14228f1fff3328f38b128b25059a23f#637a170427c62f89eb810e689d98dfa55bcdbdf2_465_465)
> I simplified the `if` statements

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5449#note_69126

[PATCH v2 0/1] MR5449: regsvr32: Implement EXE registration. - closed

[Alexandre Julliard (＠julliard)]

This merge request was closed by Alexandre Julliard.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5449

[PATCH v2 0/1] MR5449: regsvr32: Implement EXE registration.

[Alexandre Julliard (＠julliard)]

This isn't supported on Windows either:

![regsvr32](/uploads/f6356ea09b91d718caa0df02ddeae3ab/regsvr32.png)

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5449#note_87279