From b0b40d9a00bb3218408b190a387692e7ddfa028c Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Sun, 9 Dec 2007 22:31:53 +0000 Subject: 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. --- CHANGES | 5 +++++ RELEASE-NOTES | 2 +- lib/sendf.c | 2 +- 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/CHANGES b/CHANGES index 0cc616260..95c81e998 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 d69a091cd..6dbe307c1 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 ddee1d7a2..b33277ac3 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++; -- cgit v1.2.3