[PATCH 5/5] wined3d: Copy the module handle to a local variable in wined3d_cs_run().
Matteo Bruni
mbruni at codeweavers.com
Fri Sep 8 11:44:11 CDT 2017
Signed-off-by: Matteo Bruni <mbruni at codeweavers.com>
---
With the previous code there was a small window between setting the queue
as empty and the call to FreeLibraryAndExitThread() (specifically the
access to cs->wined3d_module) where the other thread could go ahead with
the destruction of the CS and the device.
dlls/wined3d/cs.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/dlls/wined3d/cs.c b/dlls/wined3d/cs.c
index 0ff0c61a45..b3b6588ff9 100644
--- a/dlls/wined3d/cs.c
+++ b/dlls/wined3d/cs.c
@@ -2569,11 +2569,16 @@ static DWORD WINAPI wined3d_cs_run(void *ctx)
unsigned int spin_count = 0;
struct wined3d_cs *cs = ctx;
enum wined3d_cs_op opcode;
+ HMODULE wined3d_module;
unsigned int poll = 0;
LONG tail;
TRACE("Started.\n");
+ /* Copy the module handle to a local variable to avoid racing with the
+ * thread freeing "cs" before the FreeLibraryAndExitThread() call. */
+ wined3d_module = cs->wined3d_module;
+
list_init(&cs->query_poll_list);
cs->thread_id = GetCurrentThreadId();
for (;;)
@@ -2621,7 +2626,7 @@ static DWORD WINAPI wined3d_cs_run(void *ctx)
cs->queue[WINED3D_CS_QUEUE_MAP].tail = cs->queue[WINED3D_CS_QUEUE_MAP].head;
cs->queue[WINED3D_CS_QUEUE_DEFAULT].tail = cs->queue[WINED3D_CS_QUEUE_DEFAULT].head;
TRACE("Stopped.\n");
- FreeLibraryAndExitThread(cs->wined3d_module, 0);
+ FreeLibraryAndExitThread(wined3d_module, 0);
}
struct wined3d_cs *wined3d_cs_create(struct wined3d_device *device)
--
2.13.5
More information about the wine-patches
mailing list