diff options
author | imp <imp@FreeBSD.org> | 2006-08-20 06:32:40 +0000 |
---|---|---|
committer | imp <imp@FreeBSD.org> | 2006-08-20 06:32:40 +0000 |
commit | 9616e66c6012dfa2a736e5776de0ddaf01e1c983 (patch) | |
tree | c6df7ddf65aef7bafdccb73a8bf8de4aff9e31a5 /bin/df | |
parent | f6de1709c1818a1f561cc0a93683fcdab75ff765 (diff) | |
download | FreeBSD-src-9616e66c6012dfa2a736e5776de0ddaf01e1c983.zip FreeBSD-src-9616e66c6012dfa2a736e5776de0ddaf01e1c983.tar.gz |
Prefer strlcpy to strncpy. In one case, this saves us from re-zeroing
data that's alreday 0. In another, it saves us from zeroing data that
will be overwritten again.
Diffstat (limited to 'bin/df')
-rw-r--r-- | bin/df/df.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/bin/df/df.c b/bin/df/df.c index 008e15c..850c02f 100644 --- a/bin/df/df.c +++ b/bin/df/df.c @@ -113,7 +113,7 @@ main(int argc, char *argv[]) memset(&totalbuf, 0, sizeof(totalbuf)); totalbuf.f_bsize = DEV_BSIZE; - strncpy(totalbuf.f_mntfromname, "total", MNAMELEN); + strlcpy(totalbuf.f_mntfromname, "total", MNAMELEN); vfslist = NULL; while ((ch = getopt(argc, argv, "abcgHhiklmnPt:")) != -1) switch (ch) { @@ -552,7 +552,7 @@ makenetvfslist(void) *str = 'n'; *(str + 1) = 'o'; for (i = 0, strptr = str + 2; i < cnt; i++, strptr++) { - strncpy(strptr, listptr[i], 32); + strlcpy(strptr, listptr[i], 32); strptr += strlen(listptr[i]); *strptr = ','; free(listptr[i]); |