diff options
| author | Todd C. Miller <millert@cvs.openbsd.org> | 2015-01-16 18:20:15 +0000 |
|---|---|---|
| committer | Todd C. Miller <millert@cvs.openbsd.org> | 2015-01-16 18:20:15 +0000 |
| commit | 86abdf609155429284902baf57e4fb5f0dec7f45 (patch) | |
| tree | 13dc6c4bc316811a8c64cba6074b091ba79055b7 /lib/libc/net | |
| parent | 221a7eb0ddf731aad0bc4cabe33ed55afcf5730c (diff) | |
Use ">", not ">=" when comparing length to HOST_NAME_MAX since
otherwise we end up needlessly replacing a NUL with a NUL.
OK deraadt@
Diffstat (limited to 'lib/libc/net')
| -rw-r--r-- | lib/libc/net/getnetent.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/libc/net/getnetent.c b/lib/libc/net/getnetent.c index fc98044a37b..b16575b64d1 100644 --- a/lib/libc/net/getnetent.c +++ b/lib/libc/net/getnetent.c @@ -1,4 +1,4 @@ -/* $OpenBSD: getnetent.c,v 1.16 2015/01/16 18:18:58 millert Exp $ */ +/* $OpenBSD: getnetent.c,v 1.17 2015/01/16 18:20:14 millert Exp $ */ /* * Copyright (c) 1983, 1993 * The Regents of the University of California. All rights reserved. @@ -86,7 +86,7 @@ again: if ((cp = strchr(p, '#')) != NULL) *cp = '\0'; net.n_name = p; - if (strlen(net.n_name) >= HOST_NAME_MAX) + if (strlen(net.n_name) > HOST_NAME_MAX) net.n_name[HOST_NAME_MAX] = '\0'; cp = strpbrk(p, " \t"); if (cp == NULL) @@ -108,7 +108,7 @@ again: } if (q < &net_aliases[MAXALIASES - 1]) { *q++ = cp; - if (strlen(cp) >= HOST_NAME_MAX) + if (strlen(cp) > HOST_NAME_MAX) cp[HOST_NAME_MAX] = '\0'; } cp = strpbrk(cp, " \t"); |