aboutsummaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
authorIngo Schwarze <schwarze@openbsd.org>2013-11-21 01:53:48 +0000
committerIngo Schwarze <schwarze@openbsd.org>2013-11-21 01:53:48 +0000
commit6de28b8b9ab60066105f904c47554cc5d6fb60b8 (patch)
tree1d303a7656234f2f2ec7e1c1132a60de7bc70c6d
parent5ffb2e3f86d7ca38fd2634b21df726b52ce35799 (diff)
downloadmandoc-6de28b8b9ab60066105f904c47554cc5d6fb60b8.tar.gz
mandoc-6de28b8b9ab60066105f904c47554cc5d6fb60b8.tar.zst
mandoc-6de28b8b9ab60066105f904c47554cc5d6fb60b8.zip
My audit of mandoc revealed one missing (unsigned char) cast
in a tolower() argument containing arbitrary char data. Thanks to deraadt@ for triggering the audit.
-rw-r--r--mandocdb.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/mandocdb.c b/mandocdb.c
index c56a2b36..a7491caf 100644
--- a/mandocdb.c
+++ b/mandocdb.c
@@ -1,4 +1,4 @@
-/* $Id: mandocdb.c,v 1.49.2.9 2013/10/10 23:43:04 schwarze Exp $ */
+/* $Id: mandocdb.c,v 1.49.2.10 2013/11/21 01:53:48 schwarze Exp $ */
/*
* Copyright (c) 2011, 2012 Kristaps Dzonsons <kristaps@bsd.lv>
* Copyright (c) 2011, 2012 Ingo Schwarze <schwarze@openbsd.org>
@@ -736,7 +736,7 @@ index_merge(const struct of *of, struct mparse *mp,
}
buf_appendb(buf, ")", 2);
for (p = buf->cp; '\0' != *p; p++)
- *p = tolower(*p);
+ *p = tolower((unsigned char)*p);
key.data = buf->cp;
key.size = buf->len;
val.data = NULL;