Author: truckman
Date: Fri May 20 07:11:04 2016
New Revision: 300284
URL: https://svnweb.freebsd.org/changeset/base/300284
Log:
MFC r299869
Use strlcpy() instead of strncpy() when copying ifname to ensure
that it is NUL terminated. Additional NUL padding is not required
for short names.
Reported by: Coverity
CID: 1009974
Modified:
stable/10/usr.sbin/route6d/route6d.c
Directory Properties:
stable/10/ (props changed)
Modified: stable/10/usr.sbin/route6d/route6d.c
==============================================================================
--- stable/10/usr.sbin/route6d/route6d.c Fri May 20 07:08:19 2016
(r300283)
+++ stable/10/usr.sbin/route6d/route6d.c Fri May 20 07:11:04 2016
(r300284)
@@ -1593,7 +1593,7 @@ ifconfig1(const char *name,
if (IN6_IS_ADDR_SITELOCAL(&sin6->sin6_addr) && !lflag)
return (-1);
ifr.ifr_addr = *sin6;
- strncpy(ifr.ifr_name, name, sizeof(ifr.ifr_name));
+ strlcpy(ifr.ifr_name, name, sizeof(ifr.ifr_name));
if (ioctl(s, SIOCGIFNETMASK_IN6, (char *)&ifr) < 0) {
syslog(LOG_INFO, "ioctl: SIOCGIFNETMASK_IN6");
return (-1);
_______________________________________________
svn-src-stable-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"
