Andrew Moise filed bug report #1847501

(http://curl.haxx.se/bug/view.cgi?id=1847501) and pointed out a memcpy()
that should be memmove() in the convert_lineends() function.

diff --git a/CHANGES b/CHANGES
index 0cc6162..95c81e9 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -7,6 +7,11 @@
                                   Changelog
 
 
+Daniel S (9 Dec 2007)
+- Andrew Moise filed bug report #1847501
+  (http://curl.haxx.se/bug/view.cgi?id=1847501) and pointed out a memcpy()
+  that should be memmove() in the convert_lineends() function.
+
 Daniel S (8 Dec 2007)
 - Renamed all internal static functions that had Curl_ prefixes to no longer
   have them. The Curl_ prefix is exclusively used for library internal global

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index d69a091..6dbe307 100644 (file)
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -53,6 +53,6 @@ advice from friends like these:
 
  Dan Fandrich, Gisle Vanem, Toby Peterson, Yang Tse, Daniel Black,
  Robin Johnson, Michal Marek, Ates Goral, Andres Garcia, Rob Crittenden,
- Emil Romanus, Alessandro Vesely, Ray Pekowski, Spacen Jasset
+ Emil Romanus, Alessandro Vesely, Ray Pekowski, Spacen Jasset, Andrew Moise
  
         Thanks! (and sorry if I forgot to mention someone)

diff --git a/lib/sendf.c b/lib/sendf.c
index ddee1d7..b33277a 100644 (file)
--- a/lib/sendf.c
+++ b/lib/sendf.c
@@ -160,7 +160,7 @@ static size_t convert_lineends(struct SessionHandle *data,
     if(*startPtr == '\n') {
       /* This block of incoming data starts with the
          previous block's LF so get rid of it */
-      memcpy(startPtr, startPtr+1, size-1);
+      memmove(startPtr, startPtr+1, size-1);
       size--;
       /* and it wasn't a bare CR but a CRLF conversion instead */
       data->state.crlf_conversions++;