[FFmpeg-devel] [PATCH] amrnb: use correct when copying lsf_r array

Benoit Fouet benoit.fouet
Sun Feb 20 18:03:46 CET 2011


On 20/02/2011 18:01, Mans Rullgard wrote:
> lsf_r is an array of int16_t, not float.
>
> Signed-off-by: Mans Rullgard<mans at mansr.com>
> ---
>   libavcodec/amrnbdec.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
> index ec896c7..2883b9c 100644
> --- a/libavcodec/amrnbdec.c
> +++ b/libavcodec/amrnbdec.c
> @@ -255,7 +255,7 @@ static void lsf2lsp_for_mode12k2(AMRContext *p, double lsp[LP_FILTER_ORDER],
>       }
>
>       if (update)
> -        memcpy(p->prev_lsf_r, lsf_r, LP_FILTER_ORDER * sizeof(float));
> +        memcpy(p->prev_lsf_r, lsf_r, LP_FILTER_ORDER * sizeof(*lsf_r));
>
>       for (i = 0; i<  LP_FILTER_ORDER; i++)
>           lsf_q[i] = lsf_r[i] * (LSF_R_FAC / 8000.0) + lsf_no_r[i] * (1.0 / 8000.0);

Looks sane, just fix your commit message.




More information about the ffmpeg-devel mailing list