svn commit: r300284 - stable/10/usr.sbin/route6d

[Don Lewis]

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);