[FFmpeg-devel] [RFC][PATCH] Avoid abort() on decoding mpeg video, just raise the error

[Reimar Döffinger]

On Wed, Dec 07, 2011 at 07:08:51PM +0100, Michael Niedermayer wrote:
> On Wed, Dec 07, 2011 at 06:15:53PM +0200, Andrey Utkin wrote:
> > Any comments are appreciated.
> > Can this patch be considered acceptable solution to make application
> > survive on broken video streams?
> > See issues
> > http://trac.videolan.org/vlc/ticket/5620
> > https://bugzilla.libav.org/show_bug.cgi?id=108
> 
> ive tried to reproduce this with latest vlc and ffmpeg but for me it
> does not abort, nor does it crash inside the ffmpegs libs
> and ffmpeg or ffplay on their own do not crash at all.
> 
> So it does not look like a ffmpeg bug to me at this point.

Neither to me, e.g. MPlayer does not have any issues with those files either.
Well, a few uninitialized data warnings from valgrind in the error
concealment code, and the error concealment does not seem to work
well at all, but that is a minor and completely different issue.