On 1/20/2018 8:00 PM, wm4 wrote: > On Sat, 20 Jan 2018 18:12:52 -0300 > James Almer <jamr...@gmail.com> wrote: > >> Signed-off-by: James Almer <jamr...@gmail.com> >> --- >> libavcodec/mediacodecdec.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c >> index 6c5d3ddd79..b360e7a7f1 100644 >> --- a/libavcodec/mediacodecdec.c >> +++ b/libavcodec/mediacodecdec.c >> @@ -258,6 +258,8 @@ static int hevc_set_extradata(AVCodecContext *avctx, >> FFAMediaFormat *format) >> } >> >> done: >> + ff_hevc_uninit_parameter_sets(&ps); >> + >> av_freep(&vps_data); >> av_freep(&sps_data); >> av_freep(&pps_data); > > Did this leak before? Or is it only needed to make it work with > patch 5/5?
It most likely leaked before. Notice how hevc parser and decoder both free the buffers, and even mediacodecdec here does it with h264. I can't test this module in any case, so someone else will have to confirm this. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel