[Ffmpeg-devel] [PATCH] h264 crashes dues to uninitialized context

Reimar Döffinger Reimar.Doeffinger
Tue Jan 16 20:58:41 CET 2007


Hello,
On Tue, Jan 16, 2007 at 03:51:10PM +0100, Michael Niedermayer wrote:
> On Tue, Jan 16, 2007 at 02:46:19PM +0100, Reimar Doeffinger wrote:
> > Hello,
> > testing the "fuzzer" samples I came about these to things that I suppose
> > are bugs: On size change, the context is freed, but context_initialized is
> > not set to 0 again.
> 
> MPV_common_end() should set it to 0

Okay.

> [...]
> > @@ -8067,6 +8072,7 @@
> >              h->inter_gb_ptr= &h->inter_gb;
> >  
> >              if(h->redundant_pic_count==0 && h->intra_gb_ptr && s->data_partitioning
> > +               && s->context_initialized
> >                 && s->hurry_up < 5
> >                 && (avctx->skip_frame < AVDISCARD_NONREF || h->nal_ref_idc)
> >                 && (avctx->skip_frame < AVDISCARD_BIDIR  || h->slice_type!=B_TYPE)
> 
> this part looks ok

Has been applied.

Greetings,
Reimar Doeffinger

P.S. my replies are a bit late because somethings wrong with my email,
this one arrived just now...




More information about the ffmpeg-devel mailing list