<div dir="ltr"><div> > I don't believe this is an actual error. If it's truly possible for<br>
> these areas to overlap (I don't think it is), then something much more<br>
> serious than using memmove instead of memcpy needs to be done about it.<br><br></div>In the C# version of this code, these two copy regions are stored in separate arrays entirely. I agree that there should be no normal way to have the memcpy overlap here (not excluding a malicious buffer overflow attack or something). Does this patch demonstrably fix the bug cited (1227580)?<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 5, 2016 at 5:00 AM,  <span dir="ltr"><<a href="mailto:opus-request@xiph.org" target="_blank">opus-request@xiph.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Send opus mailing list submissions to<br>
        <a href="mailto:opus@xiph.org">opus@xiph.org</a><br>
<br>
To subscribe or unsubscribe via the World Wide Web, visit<br>
        <a href="http://lists.xiph.org/mailman/listinfo/opus" rel="noreferrer" target="_blank">http://lists.xiph.org/mailman/listinfo/opus</a><br>
or, via email, send a message with subject or body 'help' to<br>
        <a href="mailto:opus-request@xiph.org">opus-request@xiph.org</a><br>
<br>
You can reach the person managing the list at<br>
        <a href="mailto:opus-owner@xiph.org">opus-owner@xiph.org</a><br>
<br>
When replying, please edit your Subject line so it is more specific<br>
than "Re: Contents of opus digest..."<br>
<br>
<br>
Today's Topics:<br>
<br>
   1. [PATCH] SILK: use silk_memmove for potentially overlapping<br>
      areas (Tristan Matthews)<br>
   2. Re: [PATCH] SILK: use silk_memmove for potentially<br>
      overlapping areas (Timothy B. Terriberry)<br>
<br>
<br>
----------------------------------------------------------------------<br>
<br>
Message: 1<br>
Date: Mon,  4 Jul 2016 09:18:12 -0400<br>
From: Tristan Matthews <<a href="mailto:tmatth@videolan.org">tmatth@videolan.org</a>><br>
To: <a href="mailto:opus@xiph.org">opus@xiph.org</a><br>
Cc: Tristan Matthews <<a href="mailto:tmatth@videolan.org">tmatth@videolan.org</a>><br>
Subject: [opus] [PATCH] SILK: use silk_memmove for potentially<br>
        overlapping     areas<br>
Message-ID: <<a href="mailto:1467638292-19376-1-git-send-email-tmatth@videolan.org">1467638292-19376-1-git-send-email-tmatth@videolan.org</a>><br>
<br>
Fixes CID 1227580<br>
---<br>
 silk/process_NLSFs.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/silk/process_NLSFs.c b/silk/process_NLSFs.c<br>
index c27cf03..d9123ef 100644<br>
--- a/silk/process_NLSFs.c<br>
+++ b/silk/process_NLSFs.c<br>
@@ -100,6 +100,6 @@ void silk_process_NLSFs(<br>
<br>
     } else {<br>
         /* Copy LPC coefficients for first half from second half */<br>
-        silk_memcpy( PredCoef_Q12[ 0 ], PredCoef_Q12[ 1 ], psEncC->predictLPCOrder * sizeof( opus_int16 ) );<br>
+        silk_memmove( PredCoef_Q12[ 0 ], PredCoef_Q12[ 1 ], psEncC->predictLPCOrder * sizeof( opus_int16 ) );<br>
     }<br>
 }<br>
--<br>
2.8.1<br>
<br>
<br>
<br>
------------------------------<br>
<br>
Message: 2<br>
Date: Mon, 04 Jul 2016 06:32:32 -0700<br>
From: "Timothy B. Terriberry" <<a href="mailto:tterribe@xiph.org">tterribe@xiph.org</a>><br>
To: <a href="mailto:opus@xiph.org">opus@xiph.org</a><br>
Subject: Re: [opus] [PATCH] SILK: use silk_memmove for potentially<br>
        overlapping areas<br>
Message-ID: <<a href="mailto:577A6570.4030802@xiph.org">577A6570.4030802@xiph.org</a>><br>
Content-Type: text/plain; charset=UTF-8; format=flowed<br>
<br>
Tristan Matthews wrote:<br>
> Fixes CID 1227580<br>
<br>
I don't believe this is an actual error. If it's truly possible for<br>
these areas to overlap (I don't think it is), then something much more<br>
serious than using memmove instead of memcpy needs to be done about it.<br>
<br>
<br>
------------------------------<br>
<br>
Subject: Digest Footer<br>
<br>
_______________________________________________<br>
opus mailing list<br>
<a href="mailto:opus@xiph.org">opus@xiph.org</a><br>
<a href="http://lists.xiph.org/mailman/listinfo/opus" rel="noreferrer" target="_blank">http://lists.xiph.org/mailman/listinfo/opus</a><br>
<br>
<br>
------------------------------<br>
<br>
End of opus Digest, Vol 90, Issue 4<br>
***********************************<br>
</blockquote></div><br></div>