[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.
> 
> - Hendrik

>  mpeg12.c |    1 +
>  1 file changed, 1 insertion(+)
> 7df7bf979e11e523e684e0fd5a2e3b0c60cf4971  0001-mpeg12-fixed-parsing-in-some-mpeg2-streams.patch
> From d5d1c624630fa73603e148891f8bcad8ecb2f5a1 Mon Sep 17 00:00:00 2001
> From: Hendrik Leppkes <h.leppkes at gmail.com>
> Date: Fri, 20 Jan 2012 16:46:58 +0100
> Subject: [PATCH] mpeg12: fixed parsing in some mpeg2 streams

patch applied

note, ive just posted another patch that fixes further issues ...

Thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.
-------------- 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/825c7b75/attachment.asc>