From: Daniel Stenberg Date: Sun, 9 Dec 2007 22:31:53 +0000 (+0000) Subject: Andrew Moise filed bug report #1847501 X-Git-Tag: curl-7_18_0~171 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=b0b40d9a00bb3218408b190a387692e7ddfa028c;p=thirdparty%2Fcurl.git 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 0cc616260d..95c81e998a 100644 --- 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 d69a091cd5..6dbe307c16 100644 --- 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 ddee1d7a28..b33277ac3c 100644 --- 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++;