diff options
author | truckman <truckman@FreeBSD.org> | 2016-05-15 22:31:03 +0000 |
---|---|---|
committer | truckman <truckman@FreeBSD.org> | 2016-05-15 22:31:03 +0000 |
commit | 1d8172e7625855b5f5ce6cd5f87f092e51d211de (patch) | |
tree | b91766bf2b141b666b7384c8768e9ce37ed23355 /usr.sbin/route6d | |
parent | 869fe5e5993ef8071c31c2abc96b8437aa6efab6 (diff) | |
download | FreeBSD-src-1d8172e7625855b5f5ce6cd5f87f092e51d211de.zip FreeBSD-src-1d8172e7625855b5f5ce6cd5f87f092e51d211de.tar.gz |
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
MFC after: 1 week
Diffstat (limited to 'usr.sbin/route6d')
-rw-r--r-- | usr.sbin/route6d/route6d.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/usr.sbin/route6d/route6d.c b/usr.sbin/route6d/route6d.c index 91c1d37..693493f 100644 --- a/usr.sbin/route6d/route6d.c +++ b/usr.sbin/route6d/route6d.c @@ -1592,7 +1592,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); |