Re: [Gnash-commit] gnash ChangeLog libbase/FLVParser.cpp

[Bastiaan Jacques]

On Fri, 9 May 2008, Sandro Santilli wrote:

> CVSROOT:        /sources/gnash
> Module name:    gnash
> Changes by:     Sandro Santilli <strk>    08/05/09 11:12:05
>
> Modified files:
>         .              : ChangeLog
>         libbase        : FLVParser.cpp
>
> Log message:
>                 * libbase/FLVParser.cpp: use 64 (rather then 8) padding bytes.
>                   I've no idea what that padding means and why is needed, anyway
>                   it fixes an illegal read of 33 bytes past the allocated set
>                   watching http://www.youtube.com/watch?v=P48VFXltFyM.

Some of ffmpeg's decoders use stream readers that will go past the end
of the buffer you pass it (by design) because they read 64 bytes at a
time. So we increase the buffer size to make sure they don't crash.
(This is recommended by the ffmpeg docs.)