[FFmpeg-devel] [PATCH] Fix MPEG video lowres crash

Michael Niedermayer michaelni
Thu Dec 16 02:48:17 CET 2010


On Wed, Dec 15, 2010 at 02:41:22PM +0300, Anatoly Nenashev wrote:
> Hi!
> A little fix for possible crash of MPEG video as discribed in issue  
> https://roundup.ffmpeg.org/issue2421.
> After that valgrind reports no error.

what makes you think this fix is correct
The first hunk needs explanation from you what that is supposed to do and why
the rest looks wrong. MV=0 does not need the emu code but your change looks
like it would call it. I guess theres rather a oversight related to the length
of the MC filter

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

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101216/eff5f173/attachment.pgp>



More information about the ffmpeg-devel mailing list