Re: [Mesa-dev] [PATCH] nir/clone: Free clone_state after use (to fix memory leak)

2017-11-20 Thread Tapani Pälli
On 11/20/2017 10:35 AM, Eduardo Lima Mitev wrote: Ping? This patch fixes an active memory leak. Any comments? Thanks. How to reproduce this leak? Eduardo On 11/14/2017 11:18 PM, Eduardo Lima Mitev wrote: In nir_cf_list_clone(), a state_clone object is initialized but never freed, causin

Re: [Mesa-dev] [PATCH] nir/clone: Free clone_state after use (to fix memory leak)

2017-11-20 Thread Eduardo Lima Mitev
Ping? This patch fixes an active memory leak. Any comments? Thanks. Eduardo On 11/14/2017 11:18 PM, Eduardo Lima Mitev wrote: In nir_cf_list_clone(), a state_clone object is initialized but never freed, causing a memory leak. If the remap_table argument is NULL, init_state will create a hash

[Mesa-dev] [PATCH] nir/clone: Free clone_state after use

2017-11-14 Thread Eduardo Lima Mitev
In nir_cf_list_clone(), a state_clone object is initialized but never freed, causing a memory leak. If the remap_table argument is NULL, init_state will create a hash table for it, and that's what free_clone_state is supposed to free. --- src/compiler/nir/nir_clone.c | 3 +++ 1 file changed, 3 ins