diff options
| author |   bru <bru@openbsd.org> | 2020-04-26 11:31:07 +0000 |
|---|---|---|
| committer | bru <bru@openbsd.org> | 2020-04-26 11:31:07 +0000 |
| commit | 5472a6e7c0d751517dc01d12229904d243c6e6b6 (patch) | |
| tree | da93350489006f92dbe0ea18f94c426f2ab17c5c /sys/dev/wscons | |
| parent | Remove unused assignments. (diff) | |
| download | wireguard-openbsd-5472a6e7c0d751517dc01d12229904d243c6e6b6.tar.xz wireguard-openbsd-5472a6e7c0d751517dc01d12229904d243c6e6b6.zip | |
Make explicit and enforce that WSCREEN and WSEMUL names are 0-terminated.
CID 1452993 (BUFFER_SIZE_WARNING)
CID 1453314 (BUFFER_SIZE_WARNING)
ok kettenis@
Diffstat (limited to 'sys/dev/wscons')
| -rw-r--r-- | sys/dev/wscons/wsdisplay.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/sys/dev/wscons/wsdisplay.c b/sys/dev/wscons/wsdisplay.c index e0f16514133..75edf7ba303 100644 --- a/sys/dev/wscons/wsdisplay.c +++ b/sys/dev/wscons/wsdisplay.c @@ -1,4 +1,4 @@ -/* $OpenBSD: wsdisplay.c,v 1.137 2020/04/19 15:05:14 kettenis Exp $ */ +/* $OpenBSD: wsdisplay.c,v 1.138 2020/04/26 11:31:07 bru Exp $ */ /* $NetBSD: wsdisplay.c,v 1.82 2005/02/27 00:27:52 perry Exp $ */ /* @@ -426,9 +426,9 @@ wsdisplay_getscreen(struct wsdisplay_softc *sc, if (scr == NULL) return (ENXIO); - strncpy(sd->screentype, scr->scr_dconf->scrdata->name, + strlcpy(sd->screentype, scr->scr_dconf->scrdata->name, WSSCREEN_NAME_SIZE); - strncpy(sd->emul, scr->scr_dconf->wsemul->name, WSEMUL_NAME_SIZE); + strlcpy(sd->emul, scr->scr_dconf->wsemul->name, WSEMUL_NAME_SIZE); return (0); } @@ -1265,7 +1265,7 @@ wsdisplay_internal_ioctl(struct wsdisplay_softc *sc, struct wsscreen *scr, return(EINVAL); d->nidx = sc->sc_scrdata->nscreens; - strncpy(d->name, sc->sc_scrdata->screens[d->idx]->name, + strlcpy(d->name, sc->sc_scrdata->screens[d->idx]->name, WSSCREEN_NAME_SIZE); d->ncols = sc->sc_scrdata->screens[d->idx]->ncols; d->nrows = sc->sc_scrdata->screens[d->idx]->nrows; @@ -1277,7 +1277,7 @@ wsdisplay_internal_ioctl(struct wsdisplay_softc *sc, struct wsscreen *scr, #define d ((struct wsdisplay_emultype *)data) if (wsemul_getname(d->idx) == NULL) return(EINVAL); - strncpy(d->name, wsemul_getname(d->idx), WSEMUL_NAME_SIZE); + strlcpy(d->name, wsemul_getname(d->idx), WSEMUL_NAME_SIZE); return (0); #undef d } |