Bugs item #1847501, was opened at 2007-12-09 22:41
Message generated for change (Comment added) made by bagder
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=1847501&group_id=976

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: libcurl
Group: portability problem
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: chops_at_demiurgestudios.com (andrewmoise)
Assigned to: Daniel Stenberg (bagder)
Summary: memcpy() in convert_lineends() should be memmove()

Initial Comment:
  The libcurl in CVS has this line in convert_lineends() in lib/sendf.c:

     memcpy(startPtr, startPtr+1, size-1);

  This should obviously be memmove(), since startPtr and startPtr+1 are going to overlap.
  Cheers.

----------------------------------------------------------------------

>Comment By: Daniel Stenberg (bagder)
Date: 2007-12-09 23:32

Message:
Logged In: YES
user_id=1110
Originator: NO

Good catch, thanks!

Fix committed now.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=100976&aid=1847501&group_id=976
Received on 2007-12-09