# HG changeset patch
# User Yoshiki Yazawa <yaz@cc.rim.or.jp>
# Date 1203443879 -32400
# Node ID 4fb714073ae473d730da89600a89ab8361841bcf
# Parent  7c0cee3d74d9d1f9cb2778bf83c0acaa18a0e535
fix for bug #150:
- avoid use of memcpy in these cases since memory area may overlap.
- do not perform memmove if it is not necessary.

diff -r 7c0cee3d74d9 -r 4fb714073ae4 src/madplug/decoder.c
--- a/src/madplug/decoder.c	Tue Feb 19 03:27:02 2008 -0600
+++ b/src/madplug/decoder.c	Wed Feb 20 02:57:59 2008 +0900
@@ -115,15 +115,10 @@
 
     while (1) {
         remainder = stream.bufend - stream.next_frame;
-	
-        /*
-           if (remainder >= BUFFER_SIZE)
-           {
-           printf("oh dear.. remainder = %d\n", remainder);
-           }
-         */
 
-        memcpy(buffer, stream.this_frame, remainder);
+        if(buffer != stream.this_frame && remainder)
+            memmove(buffer, stream.this_frame, remainder);
+
         len = input_get_data(info, buffer + remainder,
                              BUFFER_SIZE - remainder);
 
@@ -132,7 +127,7 @@
             break;
         }
 
-        mad_stream_buffer(&stream, buffer, len + remainder);
+        mad_stream_buffer(&stream, buffer, remainder + len);
 
         while (!fast || (fast && info->frames < N_AVERAGE_FRAMES)) {
             if (mad_header_decode(&header, &stream) == -1) {
@@ -393,12 +388,15 @@
             AUDDBG("decode: stop signaled\n");
             break;
         }
+
         if (seek_skip)
             remainder = 0;
         else {
             remainder = stream.bufend - stream.next_frame;
-            memcpy(buffer, stream.this_frame, remainder);
+            if(buffer != stream.this_frame && remainder)
+                memmove(buffer, stream.this_frame, remainder);
         }
+
         len = input_get_data(info, buffer + remainder,
                              BUFFER_SIZE - remainder);