diff options
| author | truckman <truckman@FreeBSD.org> | 2016-05-15 21:45:04 +0000 |
|---|---|---|
| committer | truckman <truckman@FreeBSD.org> | 2016-05-15 21:45:04 +0000 |
| commit | 1c108de8e62aa78dbe1a185d9bd901be14523ea7 (patch) | |
| tree | 5dc8816a66828582e630d76fc8f346a1d228d708 | |
| parent | 4d98e411343d176a82e06154becec55a04fa07ca (diff) | |
| download | FreeBSD-src-1c108de8e62aa78dbe1a185d9bd901be14523ea7.zip FreeBSD-src-1c108de8e62aa78dbe1a185d9bd901be14523ea7.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: 974852
MFC after: 1 week
| -rw-r--r-- | usr.sbin/lmcconfig/lmcconfig.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/usr.sbin/lmcconfig/lmcconfig.c b/usr.sbin/lmcconfig/lmcconfig.c index eb80dbe..65356a2 100644 --- a/usr.sbin/lmcconfig/lmcconfig.c +++ b/usr.sbin/lmcconfig/lmcconfig.c @@ -222,7 +222,7 @@ call_driver(unsigned long cmd, struct iohdr *iohdr) { int error = 0; - strncpy(iohdr->ifname, ifname, sizeof(iohdr->ifname)); + strlcpy(iohdr->ifname, ifname, sizeof(iohdr->ifname)); iohdr->cookie = NGM_LMC_COOKIE; iohdr->iohdr = iohdr; |
