[FFmpeg-devel] [PATCH] h264 - correct the check for invalid long term frame index in MMCO decode

[Jeff Downs]

On Wed, 20 Jul 2011, Michael Niedermayer wrote:

> On Mon, Jul 18, 2011 at 01:45:42PM -0400, Jeff Downs wrote:
> > On Mon, 18 Jul 2011, Michael Niedermayer wrote:
> > 
> > > On Tue, Jul 05, 2011 at 02:36:06PM -0400, Jeff Downs wrote:
> > > > The current checks on MMCO parameters prohibits a "max long term frame index
> > > > plus 1" of 16 (frame idx of 15) for the "set max long term frame index" MMCO.
> > > > Fix this off-by-one error to allow the full range of legal values.
> > > 
> > > do you have a testcase / bitstream sample for this ?
> > 
> > Yes, I do, but it is part of a licensed compliance/test set and sadly 
> > cannot share it.
> > 
> > I did verify it against the spec - the current check is not handling the 
> > boundary condition properly.
> > 
> > I don't know if you'd ever see an actual stream like this, but it was an 
> > easy enough fix.
> 
> thanks for the info & thanks for the bugfix
> patch ok

Pushed