[FFmpeg-devel] [PATCH] Fix g726 encoder

Rafaël Carré rcarre
Fri Sep 12 12:29:21 CEST 2008


On Thu, 2008-09-04 at 12:51 +0200, Michael Niedermayer wrote:
> On Thu, Sep 04, 2008 at 11:17:29AM +0200, Rafa?l Carr? wrote:
> > The g726 encoder iterates over the output buffer instead of the provided
> > samples.
> > 
> 
> > Use a big output buffer like VLC, and overwrite your memory as you see
> > fit.
> 
> I dont see how any memory could be overwritten.
> out of array reads yes, writes no

Yes my mistake it is out of array read, but the patch nevertheless stays
correct (fix the out of array read).

> > 
> > To know the size of the input buffer, the minimal frame size (in samples
> > per channel) is needed, but doesn't exist in the code.
> > 
> > I just use the same size that the reference code: 256 samples (32 ms)
> 
> I suspect this needs a update to the regression test checksums too

Is that something you can do before commit, after applying the patch ?

-- 
Rafa?l Carr? <rcarre at m2x.nl>





More information about the ffmpeg-devel mailing list