[FFmpeg-devel] [PATCH] mpeg12: fixed parsing in some mpeg2 streams

[Michael Niedermayer]

On Sat, Jan 21, 2012 at 07:57:11PM +0100, Hendrik Leppkes wrote:
> Hi,
> 
> attached patch fixes a error in the mpeg1/2 parser that caused it to
> improperly split up frames, or even swallow one at the end of the
> stream.
> Sadly, i couldn't find a way to reproduce this with one of the ff*
> tools, because the error only seemed to happen when you try to parse
> data that is already perfectly cut on frame boundaries (like from a
> MKV, or running the parser twice).
> 
> Anyhow, after reading the code it did seem kind of obvious, the other
> exit from the decoder also resets the value to 0, which makes sense -
> also need to find a new frame start when the end was found.

SEQ_END_CODEs where used with dvd still frames/pictures
please test that this change doesnt break that

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

Many that live deserve death. And some that die deserve life. Can you give
it to them? Then do not be too eager to deal out death in judgement. For
even the very wise cannot see all ends. -- Gandalf
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120122/0b1d6f0d/attachment.asc>