[FFmpeg-cvslog] lavc: fix avpacket memleak with subtitles recoding.

Clément Bœsch git at videolan.org
Mon Mar 18 22:08:17 CET 2013


ffmpeg | branch: master | Clément Bœsch <ubitux at gmail.com> | Sun Mar 17 02:30:10 2013 +0100| [4b35be3251d9dd3b9c2b86ebf8ff482ac9f5b547] | committer: Clément Bœsch

lavc: fix avpacket memleak with subtitles recoding.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=4b35be3251d9dd3b9c2b86ebf8ff482ac9f5b547
---

 libavcodec/utils.c |   10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index 94e8b57..b031132 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -2162,6 +2162,7 @@ static int recode_subtitle(AVCodecContext *avctx,
     ret = av_new_packet(&tmp, inl * UTF8_MAX_BYTES);
     if (ret < 0)
         goto end;
+    outpkt->buf  = tmp.buf;
     outpkt->data = tmp.data;
     outpkt->size = tmp.size;
     outb = outpkt->data;
@@ -2221,8 +2222,13 @@ int avcodec_decode_subtitle2(AVCodecContext *avctx, AVSubtitle *sub,
             ret = avctx->codec->decode(avctx, sub, got_sub_ptr, &pkt_recoded);
             av_assert1((ret >= 0) >= !!*got_sub_ptr &&
                        !!*got_sub_ptr >= !!sub->num_rects);
-            if (tmp.data != pkt_recoded.data)
-                av_free(pkt_recoded.data);
+            if (tmp.data != pkt_recoded.data) { // did we recode?
+                /* prevent from destroying side data from original packet */
+                pkt_recoded.side_data = NULL;
+                pkt_recoded.side_data_elems = 0;
+
+                av_free_packet(&pkt_recoded);
+            }
             sub->format = !(avctx->codec_descriptor->props & AV_CODEC_PROP_BITMAP_SUB);
             avctx->pkt = NULL;
         }



More information about the ffmpeg-cvslog mailing list