Re: [PATCH v2] perf jevents: Fix resource leak in process_mapfile() and main()

2019-10-23 Thread Yunfeng Ye
On 2019/10/16 22:25, Arnaldo Carvalho de Melo wrote: > Em Wed, Oct 16, 2019 at 09:50:17PM +0800, Yunfeng Ye escreveu: >> There are memory leaks and file descriptor resource leaks in >> process_mapfile() and main(). >> >> Fix this by adding free(), fclose() and free_arch_std_events() >> on the er

Re: [PATCH v2] perf jevents: Fix resource leak in process_mapfile() and main()

2019-10-16 Thread Arnaldo Carvalho de Melo
Em Wed, Oct 16, 2019 at 09:50:17PM +0800, Yunfeng Ye escreveu: > There are memory leaks and file descriptor resource leaks in > process_mapfile() and main(). > > Fix this by adding free(), fclose() and free_arch_std_events() > on the error paths. > > Fixes: 80eeb67fe577 ("perf jevents: Program to

[PATCH v2] perf jevents: Fix resource leak in process_mapfile() and main()

2019-10-16 Thread Yunfeng Ye
There are memory leaks and file descriptor resource leaks in process_mapfile() and main(). Fix this by adding free(), fclose() and free_arch_std_events() on the error paths. Fixes: 80eeb67fe577 ("perf jevents: Program to convert JSON file") Fixes: 3f056b66647b ("perf jevents: Make build fail on J