[FFmpeg-devel] [PATCH] lavf/matroskaenc.c: Free dyn bufs in mkv_free. Fixes memory leaks when muxing fails.
Sasi Inguva
isasi at google.com
Thu Jan 26 21:26:46 EET 2017
Signed-off-by: Sasi Inguva <isasi at google.com>
---
libavformat/matroskaenc.c | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c
index f731b678b9..88f6c647b9 100644
--- a/libavformat/matroskaenc.c
+++ b/libavformat/matroskaenc.c
@@ -393,6 +393,23 @@ static void put_xiph_size(AVIOContext *pb, int size)
* Free the members allocated in the mux context.
*/
static void mkv_free(MatroskaMuxContext *mkv) {
+ uint8_t* buf;
+ if (mkv->dyn_bc) {
+ avio_close_dyn_buf(mkv->dyn_bc, &buf);
+ av_free(buf);
+ }
+ if (mkv->info_bc) {
+ avio_close_dyn_buf(mkv->info_bc, &buf);
+ av_free(buf);
+ }
+ if (mkv->tracks_bc) {
+ avio_close_dyn_buf(mkv->tracks_bc, &buf);
+ av_free(buf);
+ }
+ if (mkv->tags_bc) {
+ avio_close_dyn_buf(mkv->tags_bc, &buf);
+ av_free(buf);
+ }
if (mkv->main_seekhead) {
av_freep(&mkv->main_seekhead->entries);
av_freep(&mkv->main_seekhead);
--
2.11.0.483.g087da7b7c-goog
More information about the ffmpeg-devel
mailing list