Re: [FFmpeg-devel] [PATCH v2] lavc/cbs_h2645_syntax_template: Fix memleak

2019-12-16 Thread James Almer
On 12/16/2019 12:00 PM, Andreas Rheinhardt wrote: > On Fri, Dec 6, 2019 at 8:22 PM Andriy Gelman > wrote: > >> From: Andriy Gelman >> >> payload_count is used to track the number of SEI payloads. It is also >> used to free the SEIs in cbs_h264_free_sei()/cbs_h265_free_sei(). >> >> Currently, pay

Re: [FFmpeg-devel] [PATCH v2] lavc/cbs_h2645_syntax_template: Fix memleak

2019-12-16 Thread Andreas Rheinhardt
On Fri, Dec 6, 2019 at 8:22 PM Andriy Gelman wrote: > From: Andriy Gelman > > payload_count is used to track the number of SEI payloads. It is also > used to free the SEIs in cbs_h264_free_sei()/cbs_h265_free_sei(). > > Currently, payload_count is set after for loop is completed. Hence if > ther

Re: [FFmpeg-devel] [PATCH v2] lavc/cbs_h2645_syntax_template: Fix memleak

2019-12-15 Thread Andriy Gelman
On Fri, 06. Dec 14:22, Andriy Gelman wrote: > From: Andriy Gelman > > payload_count is used to track the number of SEI payloads. It is also > used to free the SEIs in cbs_h264_free_sei()/cbs_h265_free_sei(). > > Currently, payload_count is set after for loop is completed. Hence if > there is an