On Wed, Sep 03, 2014 at 12:01:54AM +0200, Reimar Döffinger wrote:
> We only actually use a tiny part of it.
> 
> Signed-off-by: Reimar Döffinger <reimar.doeffin...@gmx.de>
> ---
>  libswresample/rematrix.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)

i think there a multiple loops left that can access more of the array

2 i spotted:
 for(i=0; i<64; i++){
        if(in_ch_layout & out_ch_layout & (1ULL<<i))
            matrix[i][i]= 1.0;
    }

 if(s->rematrix_volume > 0){
        for(i=0; i<SWR_CH_MAX; i++)
            for(j=0; j<SWR_CH_MAX; j++){
                s->matrix[i][j] *= s->rematrix_volume;
            }
    }


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

You can kill me, but you cannot change the truth.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to