Tristan Matthews
2016-Jul-04 13:18 UTC
[opus] [PATCH] SILK: use silk_memmove for potentially overlapping areas
Fixes CID 1227580 --- silk/process_NLSFs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/silk/process_NLSFs.c b/silk/process_NLSFs.c index c27cf03..d9123ef 100644 --- a/silk/process_NLSFs.c +++ b/silk/process_NLSFs.c @@ -100,6 +100,6 @@ void silk_process_NLSFs( } else { /* Copy LPC coefficients for first half from second half */ - silk_memcpy( PredCoef_Q12[ 0 ], PredCoef_Q12[ 1 ], psEncC->predictLPCOrder * sizeof( opus_int16 ) ); + silk_memmove( PredCoef_Q12[ 0 ], PredCoef_Q12[ 1 ], psEncC->predictLPCOrder * sizeof( opus_int16 ) ); } } -- 2.8.1
Timothy B. Terriberry
2016-Jul-04 13:32 UTC
[opus] [PATCH] SILK: use silk_memmove for potentially overlapping areas
Tristan Matthews wrote:> Fixes CID 1227580I don't believe this is an actual error. If it's truly possible for these areas to overlap (I don't think it is), then something much more serious than using memmove instead of memcpy needs to be done about it.
Possibly Parallel Threads
- opus Digest, Vol 90, Issue 4
- [PATCH 8/8] Optimize silk_NSQ_del_dec() for ARM NEON
- [PATCH 7/8] Update NSQ_LPC_BUF_LENGTH macro.
- [PATCH 8/8] Apply Neon short prediction optimization to silk_noise_shape_quantizer_del_dec.
- [Aarch64 v2 07/18] Apply Neon short prediction optimization to silk_noise_shape_quantizer_del_dec.