Re: [Mesa-dev] [PATCH 2/2] glsl: free or reuse memory allocated for TF varying

2018-11-23 Thread Tapani Pälli
Hi; On 11/23/18 10:29 AM, Gert Wollny wrote: Hello Tapani, since you wrote the original code, could you have a look at this patch? Sure. I was able to reproduce this leak with Piglit's "ext_transform_feedback-points" and fix LGTM; Reviewed-by: Tapani Pälli Many thanks, Gert Am Freita

Re: [Mesa-dev] [PATCH 2/2] glsl: free or reuse memory allocated for TF varying

2018-11-23 Thread Gert Wollny
Hello Tapani, since you wrote the original code, could you have a look at this patch? Many thanks, Gert Am Freitag, den 16.11.2018, 19:12 +0100 schrieb Gert Wollny: > From: Gert Wollny > > When a shader program is de-serialized the gl_shader_program passed > in > may actually still hold memo

[Mesa-dev] [PATCH 2/2] glsl: free or reuse memory allocated for TF varying

2018-11-16 Thread Gert Wollny
From: Gert Wollny When a shader program is de-serialized the gl_shader_program passed in may actually still hold memory allocations for the transform feedback varyings. If that is the case, free the varying names and reallocate the new storage for the names array. This fixes a memory leak: Direc