[FFmpeg-devel] [PATCH] Avoid suppressing parse errors in matroska

Reimar Döffinger Reimar.Doeffinger
Wed Feb 23 22:27:53 CET 2011


On Tue, Feb 22, 2011 at 02:00:40PM -0800, Ami Fischman wrote:
> ffmpeg can be made to loop, apparently infinitely, updating time
> counters but not video frames (@ mainline HEAD) or crash (SIGFPE
> @ ffmpeg-mt HEAD, for example) during decoding because some of
> the parsing routines silently swallow error codes.  The attached
> patch fixes this.  An example (corrupt) file that this patch
> takes from a looper/crasher to a quiet rejection is arith.webm
> attached to http://crbug.com/69218.

Neither loop nor crash is reproducible here.
Your patch seems likely to make the demuxer handle broken
files worse.
Also I cannot see anything that would cause an endless loop
(though admittedly I think it might progress through a file
very slowly).



More information about the ffmpeg-devel mailing list