diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2015-08-27 04:08:42 +0200 |
|---|---|---|
| committer | Carl Eugen Hoyos <cehoyos@ag.or.at> | 2015-10-10 00:56:06 +0200 |
| commit | 01fa1dc22914b6f01602df34b27f80ab1d007bbe (patch) | |
| tree | d1fc7a964e9100a2a802f7127c9a0e3a0eacd726 /libavformat | |
| parent | f3dfb14f6c408537ed60934dcd9bec065a582d13 (diff) | |
| download | ffmpeg-01fa1dc22914b6f01602df34b27f80ab1d007bbe.tar.gz | |
avformat/oggenc: Check segments_count for headers too
Fixes infinite loop and segfault in ogg_buffer_data()
Fixes Ticket4806
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 81a8701eb52d2b6469ae16ef442ce425388141b7)
Diffstat (limited to 'libavformat')
| -rw-r--r-- | libavformat/oggenc.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/libavformat/oggenc.c b/libavformat/oggenc.c index 1883501745..6b90a5ce80 100644 --- a/libavformat/oggenc.c +++ b/libavformat/oggenc.c @@ -255,7 +255,7 @@ static int ogg_buffer_data(AVFormatContext *s, AVStream *st, if (i == total_segments) page->granule = granule; - if (!header) { + { AVStream *st = s->streams[page->stream_index]; int64_t start = av_rescale_q(page->start_granule, st->time_base, @@ -263,10 +263,13 @@ static int ogg_buffer_data(AVFormatContext *s, AVStream *st, int64_t next = av_rescale_q(page->granule, st->time_base, AV_TIME_BASE_Q); - if (page->segments_count == 255 || - (ogg->pref_size > 0 && page->size >= ogg->pref_size) || - (ogg->pref_duration > 0 && next - start >= ogg->pref_duration)) { + if (page->segments_count == 255) { ogg_buffer_page(s, oggstream); + } else if (!header) { + if ((ogg->pref_size > 0 && page->size >= ogg->pref_size) || + (ogg->pref_duration > 0 && next - start >= ogg->pref_duration)) { + ogg_buffer_page(s, oggstream); + } } } } |