[FFmpeg-devel] [PATCH] Fix g726 encoder

Michael Niedermayer michaelni
Thu Sep 4 12:51:53 CEST 2008


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


> 
> 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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080904/9b289f90/attachment.pgp>



More information about the ffmpeg-devel mailing list