Re: Unanticipated test error

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 7/20/06, Peter Eriksen <s022018@xxxxxxxxxxxxxx> wrote:
The patch really should not change any semantics at all, since
it converts instances of

   memcpy(to, from, len);
   to[len] = 0;

into

   strlcpy(to, from, len);

I need a bit of help troubleshooting this one.  I have tried
running t0000-basic.sh using "bash -x", but that did not help
me this time.


Well, there are differences. Correct translation from memcpy
to strlcpy (aside the fact with \0 inside the string) would be
something like:

 strlcpy(to, from, len + 1);

assuming your example with memcpy. strlcpy expects size of
storage, and will never write more bytes that it was allowed to.
That'll cut off last character of the source string, unless it is
\0-terminated before the size of storage.
-
: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]