diff options
| author |   guenther <guenther@openbsd.org> | 2013-03-07 06:00:18 +0000 |
|---|---|---|
| committer | guenther <guenther@openbsd.org> | 2013-03-07 06:00:18 +0000 |
| commit | d3cd5696ade353437beeceeef0e65cce9207f2e9 (patch) | |
| tree | 1629a9d54e1d9cec08d76e87e943a69d566331bd /lib/libc | |
| parent | XL_DMACTL is a 32bit register, use the 32bit write macro. (diff) | |
| download | wireguard-openbsd-d3cd5696ade353437beeceeef0e65cce9207f2e9.tar.xz wireguard-openbsd-d3cd5696ade353437beeceeef0e65cce9207f2e9.zip | |
Oops: confstr()'s return value should include the NUL in its count
Problem pointed out by Andres Perera (andres.p (at) zoho.com)
Diffstat (limited to 'lib/libc')
| -rw-r--r-- | lib/libc/gen/confstr.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/lib/libc/gen/confstr.c b/lib/libc/gen/confstr.c index c4352648e51..0023a5e648c 100644 --- a/lib/libc/gen/confstr.c +++ b/lib/libc/gen/confstr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: confstr.c,v 1.9 2013/03/01 21:19:42 guenther Exp $ */ +/* $OpenBSD: confstr.c,v 1.10 2013/03/07 06:00:18 guenther Exp $ */ /*- * Copyright (c) 1993 * The Regents of the University of California. All rights reserved. @@ -57,7 +57,7 @@ confstr(int name, char *buf, size_t len) { switch (name) { case _CS_PATH: - return (strlcpy(buf, _PATH_STDPATH, len)); + return (strlcpy(buf, _PATH_STDPATH, len) + 1); /* no configuration-defined value */ case _CS_POSIX_V6_ILP32_OFF32_CFLAGS: @@ -111,13 +111,13 @@ confstr(int name, char *buf, size_t len) return (1); case _CS_POSIX_V7_THREADS_LDFLAGS: - return (strlcpy(buf, "-lpthread", len)); + return (strlcpy(buf, "-lpthread", len) + 1); case _CS_POSIX_V6_WIDTH_RESTRICTED_ENVS: - return (strlcpy(buf, v6_width_restricted_envs, len)); + return (strlcpy(buf, v6_width_restricted_envs, len) + 1); case _CS_POSIX_V7_WIDTH_RESTRICTED_ENVS: - return (strlcpy(buf, v7_width_restricted_envs, len)); + return (strlcpy(buf, v7_width_restricted_envs, len) + 1); default: errno = EINVAL; |