zenithsiz/ftmemsim-valgrind
1
0
Fork 0
mirror of https://github.com/Zenithsiz/ftmemsim-valgrind.git synced 2026-02-03 18:13:01 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixed

Browse Source 
Fixed bug in strncpy() and strncat() -- they were giving the right answers, but
could read the (n+1)th char, which could possibly be inaccessible.  Thanks to
Adam Gundy <arg@cyberscience.com> for spotting it.


git-svn-id: svn://svn.valgrind.org/valgrind/trunk@1557
This commit is contained in:
Nicholas Nethercote 2003-04-25 21:37:20 +00:00
parent 4e3c15f4a6
commit c9acb6f0d2
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
memcheck/mc_replace_strmem.c
View File

@ -123,8 +123,8 @@ char* strncat ( char* dst, const char* src, int n )
Int m = 0;
while (*dst) dst++;
while (*src && m++ < n) *dst++ = *src++; /* concat at most n chars */
*dst = 0; /* then add null (always) */
while (m < n && *src) { m++; *dst++ = *src++; } /* concat <= n chars */
*dst = 0; /* always add null ) */
/* This checks for overlap after copying, unavoidable without
pre-counting lengths... should be ok */
@ -164,7 +164,7 @@ char* strncpy ( char* dst, const char* src, int n )
if (is_overlap(dst, src, n))
complain3("strncpy", dst, src, n);
while (*src && m++ < n) *dst++ = *src++;
while (m < n && *src) { m++; *dst++ = *src++; }
while (m++ < n) *dst++ = 0; /* must pad remainder with nulls */
return dst_orig;