Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-08-07 07:03:06 +00:00
Code Issues Projects Releases Packages Wiki Activity

[WINESYNC] msiexec: Don't keep the custom action GUID in the main thread's stack.

Browse source 
Since it could potentially change on us during an asynchronous custom action.

Signed-off-by: Zebediah Figura <z.figura12@gmail.com>
Signed-off-by: Hans Leidekker <hans@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

wine commit id 2592690540a87498db09753a32c416cfe7f26e81 by Zebediah Figura <z.figura12@gmail.com>
This commit is contained in:
winesync 2022-03-12 23:54:32 +01:00 • committed by Mark Jansen
parent 8c067cc02c
commit 33595a42c9
No known key found for this signature in database
GPG key ID: B39240EE84BEAE8B
1 changed files with 8 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

12
base/system/msiexec/msiexec.c
View file

@ -28,6 +28,7 @@
#include <stdio.h>
#include "wine/debug.h"
#include "wine/heap.h"
#include "wine/unicode.h"
#include "initguid.h"
@ -400,20 +401,21 @@ extern UINT CDECL __wine_msi_call_dll_function(GUID *guid);
static DWORD CALLBACK custom_action_thread(void *arg)
{
GUID *guid = arg;
return __wine_msi_call_dll_function(guid);
GUID guid = *(GUID *)arg;
heap_free(arg);
return __wine_msi_call_dll_function(&guid);
}
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};
DWORD client_pid = atoiW(arg);
GUID guid, *thread_guid;
DWORD64 thread64;
WCHAR buffer[24];
HANDLE thread;
HANDLE pipe;
DWORD size;
GUID guid;
TRACE("%s\n", debugstr_w(arg));
@ -443,7 +445,9 @@ static int custom_action_server(const WCHAR *arg)
return 0;
}
thread = CreateThread(NULL, 0, custom_action_thread, &guid, 0, NULL);
thread_guid = heap_alloc(sizeof(GUID));
memcpy(thread_guid, &guid, sizeof(GUID));
thread = CreateThread(NULL, 0, custom_action_thread, thread_guid, 0, NULL);
/* give the thread handle to the client to wait on, since we might have
* to run a nested action and can't block during this one */