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.
This commit is contained in:
Daniel Stenberg 2007-12-09 22:31:53 +00:00
parent 71b105ceb1
commit b0b40d9a00
3 changed files with 7 additions and 2 deletions

View File

@ -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

View File

@ -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)

View File

@ -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++;