[FFmpeg-devel] [PATCH] snow: remove strange av_assert2

Michael Niedermayer michaelni at gmx.at
Fri Jul 3 01:04:40 CEST 2015


On Fri, Jul 03, 2015 at 12:31:31AM +0200, Andreas Cadhalpun wrote:
> It asserts that the frame linesize is larger than 37, but it can be
> smaller and decoding such frames works. Thus it is unclear what this
> assert is good for.

i think it was due to the layout of some temporary /scratch buffer

maybe in the emulated_edge_mc() call in ff_snow_pred_block() but
iam not sure
as stride becomes too small its no longer possible to store some elements
that is when they are larger

The assert should possibly be made conditional on such too large
blocks occuring when stride is too small


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- 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/20150703/134d2d15/attachment.asc>


More information about the ffmpeg-devel mailing list