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

[Michael Niedermayer]

On Tue, Apr 16, 2019 at 12:03:48AM +0200, Tomas Härdin wrote:
> mån 2019-04-15 klockan 23:03 +0200 skrev Marton Balint:
> > 
> > On Mon, 15 Apr 2019, Tomas Härdin wrote:
> > 
> > > 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()
> > 
> > I would allow 65536 bytes as well for run-in, even if that is against the 
> > standard. The MXF book for example says this:
> > 
> > run-in - a sequence of up to 65536 bytes at the front of the file that 
> > preceeds the first partition pack.
> 

> So I see. Hm.. We could report that to the publisher as an error in the
> book. But I doubt they'd do a reprint for such a minor error.

But if a reprint ever happens it is more likely to contain reported
issues than unreported ones. So i agree its a good idea to report it

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190417/fe3fa690/attachment.sig>