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_finish(&aub);
+ } else if(aub_file) {
+ fclose(aub_file);
+ }
Thanks,
Andrii.
On Fri, Oct 26, 2018 at 5:20 PM Lionel Landwerlin <
lionel.g.landwer...@intel.com> wrote:
> On 26/10/2018 15:11, asimiklit.w...@gmail.com wrote:
> > From: Andrii Simiklit <andrii.simik...@globallogic.com>
> >
> > Some memory and file descriptors are not freed/closed.
>
>
> Thanks a lot :
>
>
> Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com>
>
>
> >
> > Signed-off-by: Andrii Simiklit <andrii.simik...@globallogic.com>
> > ---
> > 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
> > --- a/src/intel/tools/error2aub.c
> > +++ b/src/intel/tools/error2aub.c
> > @@ -326,6 +326,12 @@ main(int argc, char *argv[])
> >
> > aub_write_exec(&aub, batch_addr, aub_gtt_size(&aub),
> I915_EXEC_RENDER);
> >
> > + free(out_filename);
> > + free(line);
> > + if(err_file) {
> > + fclose(err_file);
> > + }
> > + aub_file_finish(&aub);
> > return EXIT_SUCCESS;
> > }
> >
>
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
