]> git.cameronkatri.com Git - mandoc.git/blobdiff - chars.c
Don't dereference NULL pointers when formatting missing denominators,
[mandoc.git] / chars.c
diff --git a/chars.c b/chars.c
index a7b34b210574db31fa4120a5efb6ecc609018033..fe0b17dea8239449aedcf6608065e4c40c110d56 100644 (file)
--- a/chars.c
+++ b/chars.c
@@ -1,4 +1,4 @@
-/*     $Id: chars.c,v 1.60 2014/10/26 17:12:03 schwarze Exp $ */
+/*     $Id: chars.c,v 1.65 2014/10/29 00:17:43 schwarze Exp $ */
 /*
  * Copyright (c) 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2011, 2014 Ingo Schwarze <schwarze@openbsd.org>
@@ -104,18 +104,16 @@ mchars_spec2cp(const struct mchars *arg, const char *p, size_t sz)
        const struct ln *ln;
 
        ln = find(arg, p, sz);
-       return(ln != NULL ? ln->unicode : sz == 1 ? *p : -1);
+       return(ln != NULL ? ln->unicode : sz == 1 ? (unsigned char)*p : -1);
 }
 
-char
+int
 mchars_num2char(const char *p, size_t sz)
 {
        int       i;
 
-       if ((i = mandoc_strntoi(p, sz, 10)) < 0)
-               return('\0');
-
-       return(i > 0 && i < 256 && isprint(i) ? i : '\0');
+       i = mandoc_strntoi(p, sz, 10);
+       return(i >= 0 && i < 256 ? i : -1);
 }
 
 int
@@ -123,14 +121,9 @@ mchars_num2uc(const char *p, size_t sz)
 {
        int      i;
 
-       if ((i = mandoc_strntoi(p, sz, 16)) < 0)
-               return(0xFFFD);
-
-       /*
-        * XXX Code is missing here to exclude bogus ranges.
-        */
-
-       return(i <= 0x10FFFF ? i : 0xFFFD);
+       i = mandoc_strntoi(p, sz, 16);
+       assert(i >= 0 && i <= 0x10FFFF);
+       return(i);
 }
 
 const char *
@@ -149,6 +142,17 @@ mchars_spec2str(const struct mchars *arg,
        return(ln->ascii);
 }
 
+const char *
+mchars_uc2str(int uc)
+{
+       int      i;
+
+       for (i = 0; i < LINES_MAX; i++)
+               if (uc == lines[i].unicode)
+                       return(lines[i].ascii);
+       return("<?>");
+}
+
 static const struct ln *
 find(const struct mchars *tab, const char *p, size_t sz)
 {