[FFmpeg-devel] [PATCH] avcodec/rv10: check size of s->mb_width * s->mb_height

Michael Niedermayer michaelni at gmx.at
Mon Mar 2 18:35:45 CET 2015


On Mon, Mar 02, 2015 at 05:52:52PM +0100, Andreas Cadhalpun wrote:
> Hi,
> 
> encoding large frames with rv10 currently results in an assertion,
> because the number of macro blocks doesn't fit into 12 bits.
> 
> Attached patch avoids this by just not trying to write the macro
> block number in that case. The decoder can figure it out on it's
> own:

what does the binary encoder / decoder do in this case ?
changes to the encoded bitstream should be checked against a
specification or if none exists the binary decoder or encoder
to ensure that we arent generating invalid bitstreams

i suspect we dont have rv10 samples with high enough resolution but
if we do have, these could be used to ensure what we store is
correct
real video samples should be here: http://samples.ffmpeg.org/real/


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150302/dcfb464e/attachment.asc>


More information about the ffmpeg-devel mailing list