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

[Jeff Downs]

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.

	-Jeff