summaryrefslogtreecommitdiffstats
path: root/sys/kern/tty_cons.c
diff options
context:
space:
mode:
authorrobert <robert@FreeBSD.org>2002-10-17 20:03:38 +0000
committerrobert <robert@FreeBSD.org>2002-10-17 20:03:38 +0000
commit1e0cdb534a593672b1178f980abd762bd316b57d (patch)
tree0a05cf96e39ba02857dd2294138be9f6e7f278ad /sys/kern/tty_cons.c
parent5e20feda93dab6d1a2a9c89e8f6a255504b3c3a3 (diff)
downloadFreeBSD-src-1e0cdb534a593672b1178f980abd762bd316b57d.zip
FreeBSD-src-1e0cdb534a593672b1178f980abd762bd316b57d.tar.gz
Use strlcpy() instead of strncpy() to copy NUL terminated strings
for safety and consistency.
Diffstat (limited to 'sys/kern/tty_cons.c')
-rw-r--r--sys/kern/tty_cons.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/sys/kern/tty_cons.c b/sys/kern/tty_cons.c
index 40285fc..bcd3389 100644
--- a/sys/kern/tty_cons.c
+++ b/sys/kern/tty_cons.c
@@ -363,9 +363,10 @@ cn_devopen(struct cn_device *cnd, struct thread *td, int forceopen)
cnd->cnd_vp = NULL;
vn_close(vp, openflag, td->td_ucred, td);
}
- if (cnd->cnd_name[0] == '\0')
- strncpy(cnd->cnd_name, devtoname(cnd->cnd_cn->cn_dev),
+ if (cnd->cnd_name[0] == '\0') {
+ strlcpy(cnd->cnd_name, devtoname(cnd->cnd_cn->cn_dev),
sizeof(cnd->cnd_name));
+ }
snprintf(path, sizeof(path), "/dev/%s", cnd->cnd_name);
NDINIT(&nd, LOOKUP, FOLLOW, UIO_SYSSPACE, path, td);
error = vn_open(&nd, &openflag, 0);
OpenPOWER on IntegriCloud