search

StevenLOL / aforge

Automatically exported from code.google.com/p/aforge
Other
0 stars 1 forks source link

Patch for memory leak in Video.FFMPEG's VideoFileWriter::Close() #406

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
VideoFileWriter::Close() incorrectly deallocates the data->FormatContext 
structure.

According to ffmpeg's code documentation the AVFormatContext should be free'd 
with libffmpeg::avformat_free_context(AVFormatContext *s)

The entire structure end related memory is deallocated by avformat_free_context 
except for data->FormatContext->pb, which should be free'd by 
libffmpeg::avio_close(*) as shown in the attached patch

Original issue reported on code.google.com by ko...@redigo-se.net on 5 Jun 2015 at 10:15

Attachments:

GoogleCodeExporter commented 8 years ago 
Using avformat_free_context() instead of av_free() to properly free FFmpeg's 
format context.

Committed in revision 1734.

Original comment by andrew.k...@gmail.com on 5 Jun 2015 at 11:23