[PATCH vkd3d 6/7] vkd3d: Fix race condition while dumping DXBC shaders.

Józef Kucia joseph.kucia at gmail.com
Mon Oct 8 08:40:20 CDT 2018


From: Józef Kucia <jkucia at codeweavers.com>

Signed-off-by: Józef Kucia <jkucia at codeweavers.com>
---
 libs/vkd3d/state.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/libs/vkd3d/state.c b/libs/vkd3d/state.c
index b60cebf26654..1d18c31a053f 100644
--- a/libs/vkd3d/state.c
+++ b/libs/vkd3d/state.c
@@ -1203,11 +1203,14 @@ struct d3d12_pipeline_state *unsafe_impl_from_ID3D12PipelineState(ID3D12Pipeline
 
 static void dump_shader(const char *path, const char *prefix, const void *data, size_t size)
 {
-    static unsigned int shader_id = 0;
+    static int shader_id = 0;
     char filename[1024];
+    unsigned int id;
     FILE *f;
 
-    snprintf(filename, ARRAY_SIZE(filename), "%s/vkd3d-shader-%s-%u.dxbc", path, prefix, shader_id);
+    id = InterlockedIncrement(&shader_id) - 1;
+
+    snprintf(filename, ARRAY_SIZE(filename), "%s/vkd3d-shader-%s-%u.dxbc", path, prefix, id);
     if (!(f = fopen(filename, "wb")))
     {
         ERR("Failed to open %s for dumping shader.\n", filename);
@@ -1218,8 +1221,6 @@ static void dump_shader(const char *path, const char *prefix, const void *data,
         ERR("Failed to write shader to %s.\n", filename);
     if (fclose(f))
         ERR("Failed to close stream %s.\n", filename);
-
-    ++shader_id;
 }
 
 static void dump_shader_stage(VkShaderStageFlagBits stage, const void *data, size_t size)
-- 
2.18.0




More information about the wine-devel mailing list