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

2018-11-01 Thread Lionel Landwerlin
Pushed to master. Thanks Andrey! On 31/10/2018 11:48, andrey simiklit wrote: Hello, Sorry, one more point here. Could I ask you to push it, because I don't have a push rights :-) Thanks a lot, Andrii. On Fri, Oct 26, 2018 at 6:09 PM Lionel Landwerlin mailto:lionel.g.landwer...@intel.com>> w

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

2018-10-31 Thread andrey simiklit
Hello, Sorry, one more point here. Could I ask you to push it, because I don't have a push rights :-) Thanks a lot, Andrii. On Fri, Oct 26, 2018 at 6:09 PM Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > On 26/10/2018 15:29, asimiklit.w...@gmail.com wrote: > > From: Andrii Simiklit

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

2018-10-26 Thread Lionel Landwerlin
On 26/10/2018 15:29, asimiklit.w...@gmail.com wrote: From: Andrii Simiklit Some memory and file descriptors are not freed/closed. v2: fixed case where we skipped the 'aub' variable initialization Signed-off-by: Andrii Simiklit Still : Reviewed-by: Lionel Landwerlin Thanks! --- sr

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

2018-10-26 Thread asimiklit . work
From: Andrii Simiklit Some memory and file descriptors are not freed/closed. v2: fixed case where we skipped the 'aub' variable initialization Signed-off-by: Andrii Simiklit --- src/intel/tools/error2aub.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/src/intel/tools/error2a