From 1d6233d315dc0af552b16e5b4d14084fbeb9fbb5 Mon Sep 17 00:00:00 2001 From: Aurelien Jacobs Date: Fri, 31 Dec 2010 00:01:37 +0000 Subject: [PATCH] use avsubtitle_free() instead of re-implementing it partially (fix a memleak) Originally committed as revision 26175 to svn://svn.ffmpeg.org/ffmpeg/trunk --- ffmpeg.c | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index 94919814bc..87f6d99d82 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -1747,15 +1747,7 @@ static int output_packet(AVInputStream *ist, int ist_index, av_free(buffer_to_free); /* XXX: allocate the subtitles in the codec ? */ if (subtitle_to_free) { - if (subtitle_to_free->rects != NULL) { - for (i = 0; i < subtitle_to_free->num_rects; i++) { - av_freep(&subtitle_to_free->rects[i]->pict.data[0]); - av_freep(&subtitle_to_free->rects[i]->pict.data[1]); - av_freep(&subtitle_to_free->rects[i]); - } - av_freep(&subtitle_to_free->rects); - } - subtitle_to_free->num_rects = 0; + avsubtitle_free(subtitle_to_free); subtitle_to_free = NULL; } }