aboutsummaryrefslogtreecommitdiffstats
path: root/ffmpeg.c
diff options
context:
space:
mode:
authorMarton Balint <cus@passwd.hu>2013-11-23 17:27:22 +0100
committerMarton Balint <cus@passwd.hu>2013-11-28 21:30:03 +0100
commit4d6f2ff524afc01f2af30677ca271a6387cf8657 (patch)
tree402913f894299ba3fb183bdf66a8a871f51daffd /ffmpeg.c
parent3e396ca8eda473d619ce5ffc1e01950c978194e8 (diff)
downloadffmpeg-4d6f2ff524afc01f2af30677ca271a6387cf8657.tar.gz
ffmpeg: free empty subtitles as well in transcode_subtitles
Even if it does not matter at the moment, because subtitles with num_rect == 0 have no memory allocated, this is how we expect the users to use the API, a returned AVSubtitle should be freed with avsubtitle_free. Signed-off-by: Marton Balint <cus@passwd.hu> Reviewed-by: Nicolas George <george@nsup.org>
Diffstat (limited to 'ffmpeg.c')
-rw-r--r--ffmpeg.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/ffmpeg.c b/ffmpeg.c
index bbd30190cb..105efbcb22 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -1828,7 +1828,7 @@ static int transcode_subtitles(InputStream *ist, AVPacket *pkt, int *got_output)
sub2video_update(ist, &subtitle);
if (!subtitle.num_rects)
- return ret;
+ goto out;
for (i = 0; i < nb_output_streams; i++) {
OutputStream *ost = output_streams[i];
@@ -1839,6 +1839,7 @@ static int transcode_subtitles(InputStream *ist, AVPacket *pkt, int *got_output)
do_subtitle_out(output_files[ost->file_index]->ctx, ost, ist, &subtitle);
}
+out:
avsubtitle_free(&subtitle);
return ret;
}