[FFmpeg-devel] [PATCH] avformat/oggparseogm: sync avctx w/ codecpar

Carl Eugen Hoyos ceffmpeg at gmail.com
Fri Feb 8 22:51:11 EET 2019


2019-02-08 21:07 GMT+01:00, James Almer <jamrial at gmail.com>:
> On 2/8/2019 12:17 AM, Chris Cunningham wrote:
>> On Wed, Feb 6, 2019 at 6:03 PM James Almer <jamrial at gmail.com> wrote:
>>
>>> Can a valid ogm stream contain more than one header packet?
>>
>>
>> Looking at ogg_packet oggdec.c, we read headers until hitting the first
>> non-header (counted in ogg stream field nb_headers), so multiple headers
>> per stream is probably ok.
>
> Probably ok as in our code currently allows it? Because if the spec
> disallows it, then that should be changed.

If our current code allows it, it should not be changed without
a good reason and certainly not in a patch that fixes an abort.

Carl Eugen


More information about the ffmpeg-devel mailing list