diff options
author | Marton Balint <cus@passwd.hu> | 2013-11-23 17:27:22 +0100 |
---|---|---|
committer | Marton Balint <cus@passwd.hu> | 2013-11-28 21:30:03 +0100 |
commit | 4d6f2ff524afc01f2af30677ca271a6387cf8657 (patch) | |
tree | 402913f894299ba3fb183bdf66a8a871f51daffd | |
parent | 3e396ca8eda473d619ce5ffc1e01950c978194e8 (diff) | |
download | ffmpeg-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>
-rw-r--r-- | ffmpeg.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -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; } |