diff options
| author |   moritz <moritz@openbsd.org> | 2007-01-03 20:02:10 +0000 |
|---|---|---|
| committer | moritz <moritz@openbsd.org> | 2007-01-03 20:02:10 +0000 |
| commit | 474fc5f5d01fba8e2e71f44358d02322bf251e9d (patch) | |
| tree | 509c0c71ffa1dd5a993026d4bf47f3637183ef19 | |
| parent | fix CVS_CLIENT_LOG logging, we were missing parts (diff) | |
| download | wireguard-openbsd-474fc5f5d01fba8e2e71f44358d02322bf251e9d.tar.xz wireguard-openbsd-474fc5f5d01fba8e2e71f44358d02322bf251e9d.zip | |
Fix format string bugs by using strlcpy() instead of
snprintf() and fix truncation checks.
ok miod@ deraadt@
| -rw-r--r-- | usr.sbin/eeprom/eehandlers.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/usr.sbin/eeprom/eehandlers.c b/usr.sbin/eeprom/eehandlers.c index e304092c0e4..0a2c6c25214 100644 --- a/usr.sbin/eeprom/eehandlers.c +++ b/usr.sbin/eeprom/eehandlers.c @@ -1,4 +1,4 @@ -/* $OpenBSD: eehandlers.c,v 1.12 2004/08/01 18:32:17 deraadt Exp $ */ +/* $OpenBSD: eehandlers.c,v 1.13 2007/01/03 20:02:10 moritz Exp $ */ /* $NetBSD: eehandlers.c,v 1.2 1996/02/28 01:13:22 thorpej Exp $ */ /*- @@ -398,9 +398,8 @@ ee_diagpath(struct keytabent *ktent, char *arg) bzero(path, sizeof(path)); if (arg) { - if (strlen(arg) > sizeof(path)) + if (strlcpy(path, arg, sizeof(path)) >= sizeof(path)) BARF(ktent); - snprintf(path, sizeof path, arg); if (doio(ktent, (u_char *)&path[0], sizeof(path), IO_WRITE)) FAILEDWRITE(ktent); } else @@ -423,11 +422,11 @@ ee_banner(struct keytabent *ktent, char *arg) bzero(string, sizeof(string)); if (arg) { - if (strlen(arg) > sizeof(string)) - BARF(ktent); if (*arg != '\0') { enable = EE_TRUE; - snprintf(string, sizeof string, arg); + if (strlcpy(string, arg, sizeof(string)) >= + sizeof(string)) + BARF(ktent); if (doio(ktent, (u_char *)string, sizeof(string), IO_WRITE)) FAILEDWRITE(ktent); |