On Sun, Jul 05, 2015 at 09:11:44PM +0200, Andreas Cadhalpun wrote: > On 03.07.2015 01:04, Michael Niedermayer wrote: > > 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 > > I couldn't find a problem with the layout of the tmp/scratchbuf buffer > for small strides. > > > as stride becomes too small its no longer possible to store some elements > > that is when they are larger > > Can you explain how elements can be too larger to fit?
emulated_edge_mc() writes a block of width x height, if stride < width this will not work as intended iam quite unsure if this is the intended thing for the assert to check, i hthik there was more code that possibly was fixed [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Avoid a single point of failure, be that a person or equipment.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
