Re: [Mesa-dev] [PATCH] intel/tools: fix resource leak

2018-10-26 Thread andrey simiklit
Hello, Thanks a lot for reviewing and for quick reply. Sorry, I forgot ( I thought that it impossible to have the feedback in a 10 minutes on Friday :-) ) to update my inbox and sent version v2, I guess that I fix small issue in v2: - aub_file_finish(&aub); + if(aub.file) { + aub_file_fin

Re: [Mesa-dev] [PATCH] intel/tools: fix resource leak

2018-10-26 Thread Lionel Landwerlin
On 26/10/2018 15:11, asimiklit.w...@gmail.com wrote: From: Andrii Simiklit Some memory and file descriptors are not freed/closed. Thanks a lot : Reviewed-by: Lionel Landwerlin Signed-off-by: Andrii Simiklit --- src/intel/tools/error2aub.c | 6 ++ 1 file changed, 6 insertions(+

[Mesa-dev] [PATCH] intel/tools: fix resource leak

2018-10-26 Thread asimiklit . work
From: Andrii Simiklit Some memory and file descriptors are not freed/closed. Signed-off-by: Andrii Simiklit --- src/intel/tools/error2aub.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/intel/tools/error2aub.c b/src/intel/tools/error2aub.c index 8a23d5ef1e..451fd8c74a 100644 --