[Mesa-dev] [PATCH shader-db 1/2] run: Set current_shader_names[i] to NULL before freeing it.

Kenneth Graunke kenneth at whitecape.org
Tue Feb 28 01:53:20 UTC 2017


current_shader_names[i] is set to shader_test[i].filename.  Before we
free that, we should NULL out the pointer (also indicating the current
thread is done with this shader).  That way, the crash handler won't
print garbage when trying to list what threads were doing.
---
 run.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/run.c b/run.c
index 8c123d5..a675a77 100644
--- a/run.c
+++ b/run.c
@@ -724,6 +724,8 @@ main(int argc, char **argv)
             }
             shaders_compiled += num_shaders;
 
+            current_shader_names[omp_get_thread_num()] = NULL;
+
             free(shader);
             free(shader_test[i].filename);
 
-- 
2.11.1



More information about the mesa-dev mailing list