https://git.reactos.org/?p=reactos.git;a=commitdiff;h=cdb9abd62581e02b75bcb…
commit cdb9abd62581e02b75bcb5b05595a5a693a981d8
Author:     winesync <ros-dev(a)reactos.org>
AuthorDate: Sun Mar 13 19:08:57 2022 +0100
Commit:     Mark Jansen <mark.jansen(a)reactos.org>
CommitDate: Sun Mar 20 19:28:33 2022 +0100
    [WINESYNC] msiexec: Use wide-char string literals.
    Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org>
    Signed-off-by: Hans Leidekker <hans(a)codeweavers.com>
    Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>
    wine commit id 64709fe7e9c629f476fb0dd7b62480e868b0174c by Michael Stefaniuc
<mstefani(a)winehq.org>
---
 base/system/msiexec/msiexec.c | 37 +++++++++----------------------------
 1 file changed, 9 insertions(+), 28 deletions(-)
diff --git a/base/system/msiexec/msiexec.c b/base/system/msiexec/msiexec.c
index 17d4bf87cdd..12ff7e7dc4e 100644
--- a/base/system/msiexec/msiexec.c
+++ b/base/system/msiexec/msiexec.c
@@ -46,19 +46,6 @@ struct string_list
        WCHAR str[1];
 };
-static const WCHAR ActionAdmin[] = {
-
'A','C','T','I','O','N','=','A','D','M','I','N',0
};
-static const WCHAR RemoveAll[] = {
-
'R','E','M','O','V','E','=','A','L','L',0
};
-
-static const WCHAR InstallRunOnce[] = {
-
'S','o','f','t','w','a','r','e','\\',
-
'M','i','c','r','o','s','o','f','t','\\',
-
'W','i','n','d','o','w','s','\\',
-
'C','u','r','r','e','n','t','V','e','r','s','i','o','n','\\',
-
'I','n','s','t','a','l','l','e','r','\\',
-
'R','u','n','O','n','c','e','E','n','t','r','i','e','s',0};
-
 static void ShowUsage(int ExitCode)
 {
     WCHAR msiexec_version[40];
@@ -339,8 +326,6 @@ static DWORD DoDllUnregisterServer(LPCWSTR DllName)
 static DWORD DoRegServer(void)
 {
-    static const WCHAR msiserverW[] =
{'M','S','I','S','e','r','v','e','r',0};
-    static const WCHAR msiexecW[] =
{'\\','m','s','i','e','x','e','c','
','/','V',0};
     SC_HANDLE scm, service;
     WCHAR path[MAX_PATH+12];
     DWORD len, ret = 0;
@@ -351,8 +336,8 @@ static DWORD DoRegServer(void)
         return 1;
     }
     len = GetSystemDirectoryW(path, MAX_PATH);
-    lstrcpyW(path + len, msiexecW);
-    if ((service = CreateServiceW(scm, msiserverW, msiserverW, GENERIC_ALL,
+    lstrcpyW(path + len, L"\\msiexec /V");
+    if ((service = CreateServiceW(scm, L"MSIServer", L"MSIServer",
GENERIC_ALL,
                                   SERVICE_WIN32_SHARE_PROCESS, SERVICE_DEMAND_START,
                                   SERVICE_ERROR_NORMAL, path, NULL, NULL, NULL, NULL,
NULL)))
     {
@@ -369,7 +354,6 @@ static DWORD DoRegServer(void)
 static DWORD DoUnregServer(void)
 {
-    static const WCHAR msiserverW[] =
{'M','S','I','S','e','r','v','e','r',0};
     SC_HANDLE scm, service;
     DWORD ret = 0;
@@ -378,7 +362,7 @@ static DWORD DoUnregServer(void)
         fprintf(stderr, "Failed to open service control manager\n");
         return 1;
     }
-    if ((service = OpenServiceW(scm, msiserverW, DELETE)))
+    if ((service = OpenServiceW(scm, L"MSIServer", DELETE)))
     {
         if (!DeleteService(service))
         {
@@ -409,7 +393,6 @@ static DWORD CALLBACK custom_action_thread(void *arg)
 static int custom_action_server(const WCHAR *arg)
 {
-    static const WCHAR pipe_name[] =
{'\\','\\','.','\\','p','i','p','e','\\','m','s','i','c','a','_','%','x','_','%','d',0};
     GUID guid, *thread_guid;
     DWORD64 thread64;
     WCHAR buffer[24];
@@ -425,7 +408,7 @@ static int custom_action_server(const WCHAR *arg)
         return 1;
     }
-    swprintf(buffer, ARRAY_SIZE(buffer), pipe_name, client_pid, sizeof(void *) * 8);
+    swprintf(buffer, ARRAY_SIZE(buffer), L"\\\\.\\pipe\\msica_%x_%d",
client_pid, (int)(sizeof(void *) * 8));
     pipe = CreateFileW(buffer, GENERIC_READ | GENERIC_WRITE, 0, NULL, OPEN_EXISTING, 0,
NULL);
     if (pipe == INVALID_HANDLE_VALUE)
     {
@@ -577,7 +560,8 @@ static BOOL process_args_from_reg( const WCHAR *ident, int *pargc,
WCHAR ***parg
        WCHAR *buf;
        BOOL ret = FALSE;
-       r = RegOpenKeyW(HKEY_LOCAL_MACHINE, InstallRunOnce, &hkey);
+       r = RegOpenKeyW(HKEY_LOCAL_MACHINE,
+
L"Software\\Microsoft\\Windows\\CurrentVersion\\Installer\\RunOnceEntries",
&hkey);
        if(r != ERROR_SUCCESS)
                return FALSE;
        r = RegQueryValueExW(hkey, ident, 0, &type, 0, &sz);
@@ -700,7 +684,7 @@ int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR
lpCmdLine
                                ShowUsage(1);
                        WINE_TRACE("argvW[%d] = %s\n", i,
wine_dbgstr_w(argvW[i]));
                        PackageName = argvW[i];
-                       StringListAppend(&property_list, ActionAdmin);
+                       StringListAppend(&property_list, L"ACTION=ADMIN");
                        WINE_FIXME("Administrative installs are not currently
supported\n");
                }
                else if(msi_option_prefix(argvW[i], "f"))
@@ -783,7 +767,7 @@ int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR
lpCmdLine
                                PackageName = argvW[i];
                        }
                        WINE_TRACE("PackageName = %s\n",
wine_dbgstr_w(PackageName));
-                       StringListAppend(&property_list, RemoveAll);
+                       StringListAppend(&property_list, L"REMOVE=ALL");
                }
                else if(msi_option_prefix(argvW[i], "j"))
                {
@@ -996,11 +980,8 @@ int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
LPSTR lpCmdLine
                }
                 else if(msi_option_equal(argvW[i], "passive"))
                 {
-                    static const WCHAR rebootpromptW[] =
-
{'R','E','B','O','O','T','P','R','O','M','P','T','=','"','S','"',0};
-
                     InstallUILevel =
INSTALLUILEVEL_BASIC|INSTALLUILEVEL_PROGRESSONLY|INSTALLUILEVEL_HIDECANCEL;
-                    StringListAppend(&property_list, rebootpromptW);
+                    StringListAppend(&property_list,
L"REBOOTPROMPT=\"S\"");
                 }
                else if(msi_option_equal(argvW[i], "y"))
                {