[FFmpeg-devel] [PATCH] mxfdec: Constrain run-in to 64k

[Carl Eugen Hoyos]

2019-04-16 0:00 GMT+02:00, Tomas Härdin <tjoppen at acc.umu.se>:
> mån 2019-04-15 klockan 12:40 +0200 skrev Carl Eugen Hoyos:
>> > 2019-04-15 10:30 GMT+02:00, Tomas Härdin <tjoppen at acc.umu.se>:
>> > This isn't likely to be a huge problem, but it allows us to reason more
>> > about run-in. It also exposes my gripe about klv_read_packet() using
>> > mxf_read_sync()
>>
>> Does this patch have an effect on one of our samples?
>
> It passes FATE if that's what you mean.

No.

> The intent is rather to pre-emptively stop the ability for new
> broken muxers to pop up, which would end up straying
> from the spec because mxfdec.c is being too forgiving

As such, and if I don't misunderstand, this sounds to me
like an explanation why this patch should not be pushed.

Carl Eugen