[FFmpeg-devel] [libav-devel] [PATCH 4/4] Reset the internal state when aborting key frames header parsing in vp6 decoder.

[Michael Niedermayer]

On Sat, Sep 24, 2011 at 04:19:25PM +0200, Laurent Aimar wrote:
> On Sat, Sep 24, 2011 at 04:16:41PM +0200, fenrir at elivagar.org wrote:
> > From: Laurent Aimar <fenrir at videolan.org>
> > 
> > It prevents leaving the state only half initialized.
> > ---
> >  libavcodec/vp6.c |    6 ++++--
> >  1 files changed, 4 insertions(+), 2 deletions(-)
> > 
> > diff --git a/libavcodec/vp6.c b/libavcodec/vp6.c
> > index 6928a91..3cac5dc 100644
> > --- a/libavcodec/vp6.c
> > +++ b/libavcodec/vp6.c
> > @@ -57,7 +57,6 @@ static int vp6_parse_header(VP56Context *s, const uint8_t *buf, int buf_size,
> >  
> >      s->framep[VP56_FRAME_CURRENT]->key_frame = !(buf[0] & 0x80);
> >      ff_vp56_init_dequant(s, (buf[0] >> 1) & 0x3F);
> > -
> >      if (s->framep[VP56_FRAME_CURRENT]->key_frame) {
> >          sub_version = buf[1] >> 3;
> >          if (sub_version > 8)
>  I have attached a version without this change.
> 
> -- 
> fenrir

>  vp6.c |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 6a5e6ecb14037fb880e6359d899942784151aa06  0004-Reset-the-internal-state-when-aborting-key-frames-he.patch
> From 65421b5cb506536bcc9d6a5b6a53c2d29b0583b2 Mon Sep 17 00:00:00 2001
> From: Laurent Aimar <fenrir at videolan.org>
> Date: Fri, 23 Sep 2011 22:36:11 +0200
> Subject: [PATCH 4/4] Reset the internal state when aborting key frames header parsing in vp6 decoder.
> 
> It prevents leaving the state only half initialized.

applied, thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.
-------------- 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/20110924/038e555f/attachment.asc>