diff options
| author |   schwarze <schwarze@openbsd.org> | 2013-11-21 01:47:10 +0000 |
|---|---|---|
| committer | schwarze <schwarze@openbsd.org> | 2013-11-21 01:47:10 +0000 |
| commit | da448cac479f3934f44f5fef246c97b4a2b8cf3e (patch) | |
| tree | 360601116a28d4b8b427111db581a099143e1e26 /usr.bin/mandoc/manpath.c | |
| parent | remove #define b_errcnt b_resid. nothing uses it, and nothing should. (diff) | |
| download | wireguard-openbsd-da448cac479f3934f44f5fef246c97b4a2b8cf3e.tar.xz wireguard-openbsd-da448cac479f3934f44f5fef246c97b4a2b8cf3e.zip | |
My audit of mandoc revealed two missing (unsigned char) casts
in isspace() and tolower() arguments containing arbitrary char data.
Thanks to deraadt@ for triggering the audit.
Diffstat (limited to 'usr.bin/mandoc/manpath.c')
| -rw-r--r-- | usr.bin/mandoc/manpath.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr.bin/mandoc/manpath.c b/usr.bin/mandoc/manpath.c index a22c23e3113..3e7a45ba423 100644 --- a/usr.bin/mandoc/manpath.c +++ b/usr.bin/mandoc/manpath.c @@ -1,4 +1,4 @@ -/* $Id: manpath.c,v 1.6 2013/05/29 23:15:11 schwarze Exp $ */ +/* $Id: manpath.c,v 1.7 2013/11/21 01:47:10 schwarze Exp $ */ /* * Copyright (c) 2011 Ingo Schwarze <schwarze@openbsd.org> * Copyright (c) 2011 Kristaps Dzonsons <kristaps@bsd.lv> @@ -159,7 +159,7 @@ manpath_manconf(struct manpaths *dirs, const char *file) if (strncmp(MAN_CONF_KEY, p, keysz)) continue; p += keysz; - while (isspace(*p)) + while (isspace((unsigned char)*p)) p++; if ('\0' == *p) continue; |