diff options
| author |   millert <millert@openbsd.org> | 2014-12-03 17:00:15 +0000 |
|---|---|---|
| committer | millert <millert@openbsd.org> | 2014-12-03 17:00:15 +0000 |
| commit | 4584f6b941b1cc8c899a5983f0bce79acab5c396 (patch) | |
| tree | c094ea82b146e56af115c9109243b960903ebe61 /usr.sbin/syslogd/syslogd.c | |
| parent | Fix buffer overflow in .PS parsing. Found by afl and based on a (diff) | |
| download | wireguard-openbsd-4584f6b941b1cc8c899a5983f0bce79acab5c396.tar.xz wireguard-openbsd-4584f6b941b1cc8c899a5983f0bce79acab5c396.zip | |
Fix off by one in check for socket path too long. From Nicolas Bedos.
Diffstat (limited to 'usr.sbin/syslogd/syslogd.c')
| -rw-r--r-- | usr.sbin/syslogd/syslogd.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr.sbin/syslogd/syslogd.c b/usr.sbin/syslogd/syslogd.c index 9548afe08db..1b739ae3eb5 100644 --- a/usr.sbin/syslogd/syslogd.c +++ b/usr.sbin/syslogd/syslogd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: syslogd.c,v 1.131 2014/11/26 18:34:52 millert Exp $ */ +/* $OpenBSD: syslogd.c,v 1.132 2014/12/03 17:00:15 millert Exp $ */ /* * Copyright (c) 1983, 1988, 1993, 1994 @@ -1815,7 +1815,7 @@ unix_socket(char *path, int type, mode_t mode) memset(&s_un, 0, sizeof(s_un)); s_un.sun_family = AF_UNIX; - if (strlcpy(s_un.sun_path, path, sizeof(s_un.sun_path)) > + if (strlcpy(s_un.sun_path, path, sizeof(s_un.sun_path)) >= sizeof(s_un.sun_path)) { snprintf(errbuf, sizeof(errbuf), "socket path too long: %s", path); |