]> git.cameronkatri.com Git - mandoc.git/blobdiff - chars.c
Avoid out-of-bounds read access before the beginning of the
[mandoc.git] / chars.c
diff --git a/chars.c b/chars.c
index 2653983256671ccc22cf116dcbf67b88086d7a06..6b5eba953728f1aa926d246902e4a3bdc4445d24 100644 (file)
--- a/chars.c
+++ b/chars.c
@@ -1,7 +1,7 @@
-/*     $Id: chars.c,v 1.36 2011/04/29 22:18:12 kristaps Exp $ */
+/*     $Id: chars.c,v 1.66 2015/02/17 20:37:16 schwarze Exp $ */
 /*
- * Copyright (c) 2009, 2010 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2011 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
+ * Copyright (c) 2011, 2014 Ingo Schwarze <schwarze@openbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
-#ifdef HAVE_CONFIG_H
 #include "config.h"
-#endif
+
+#include <sys/types.h>
 
 #include <assert.h>
-#include <stdio.h>
+#include <ctype.h>
 #include <stdlib.h>
 #include <string.h>
 
 #include "mandoc.h"
+#include "mandoc_aux.h"
+#include "libmandoc.h"
 
 #define        PRINT_HI         126
 #define        PRINT_LO         32
@@ -34,20 +36,12 @@ struct      ln {
        const char       *code;
        const char       *ascii;
        int               unicode;
-       int               type;
-#define        CHARS_CHAR       (1 << 0)
-#define        CHARS_STRING     (1 << 1)
-#define CHARS_BOTH      (CHARS_CHAR | CHARS_STRING)
 };
 
-#define        LINES_MAX         353
+#define        LINES_MAX         332
 
 #define CHAR(in, ch, code) \
-       { NULL, (in), (ch), (code), CHARS_CHAR },
-#define STRING(in, ch, code) \
-       { NULL, (in), (ch), (code), CHARS_STRING },
-#define BOTH(in, ch, code) \
-       { NULL, (in), (ch), (code), CHARS_BOTH },
+       { NULL, (in), (ch), (code) },
 
 #define        CHAR_TBL_START    static struct ln lines[LINES_MAX] = {
 #define        CHAR_TBL_END      };
@@ -55,13 +49,12 @@ struct      ln {
 #include "chars.in"
 
 struct mchars {
-       enum mcharst      type;
        struct ln       **htab;
 };
 
-static inline int        match(const struct ln *,
-                               const char *, size_t, int);
-static const struct ln  *find(struct mchars *, const char *, size_t, int);
+static const struct ln  *find(const struct mchars *,
+                               const char *, size_t);
+
 
 void
 mchars_free(struct mchars *arg)
@@ -72,7 +65,7 @@ mchars_free(struct mchars *arg)
 }
 
 struct mchars *
-mchars_init(enum mcharst type)
+mchars_alloc(void)
 {
        struct mchars    *tab;
        struct ln       **htab;
@@ -82,12 +75,11 @@ mchars_init(enum mcharst type)
        /*
         * Constructs a very basic chaining hashtable.  The hash routine
         * is simply the integral value of the first character.
-        * Subsequent entries are chained in the order they're processed
-        * (they're in-line re-ordered during lookup).
+        * Subsequent entries are chained in the order they're processed.
         */
 
        tab = mandoc_malloc(sizeof(struct mchars));
-       htab = mandoc_calloc(PRINT_HI - PRINT_LO + 1, sizeof(struct ln **));
+       htab = mandoc_calloc(PRINT_HI - PRINT_LO + 1, sizeof(struct ln *));
 
        for (i = 0; i < LINES_MAX; i++) {
                hash = (int)lines[i].code[0] - PRINT_LO;
@@ -103,146 +95,81 @@ mchars_init(enum mcharst type)
        }
 
        tab->htab = htab;
-       tab->type = type;
        return(tab);
 }
 
-
-/* 
- * Special character to Unicode codepoint.
- */
 int
-mchars_spec2cp(struct mchars *arg, const char *p, size_t sz)
+mchars_spec2cp(const struct mchars *arg, const char *p, size_t sz)
 {
        const struct ln *ln;
 
-       ln = find(arg, p, sz, CHARS_CHAR);
-       if (NULL == ln)
-               return(-1);
-       return(ln->unicode);
+       ln = find(arg, p, sz);
+       return(ln != NULL ? ln->unicode : sz == 1 ? (unsigned char)*p : -1);
 }
 
-
-/* 
- * Reserved word to Unicode codepoint.
- */
 int
-mchars_res2cp(struct mchars *arg, const char *p, size_t sz)
+mchars_num2char(const char *p, size_t sz)
 {
-       const struct ln *ln;
+       int       i;
 
-       ln = find(arg, p, sz, CHARS_STRING);
-       if (NULL == ln)
-               return(-1);
-       return(ln->unicode);
+       i = mandoc_strntoi(p, sz, 10);
+       return(i >= 0 && i < 256 ? i : -1);
 }
 
-
-/*
- * Numbered character to literal character,
- * represented as a null-terminated string for additional safety.
- */
-const char *
-mchars_num2char(const char *p, size_t sz)
+int
+mchars_num2uc(const char *p, size_t sz)
 {
-       int               i;
-       static char       c[2];
+       int      i;
 
-       if (sz > 3)
-               return(NULL);
-       i = atoi(p);
-       if (i < 0 || i > 255)
-               return(NULL);
-       c[0] = (char)i;
-       c[1] = '\0';
-       return(c);
+       i = mandoc_strntoi(p, sz, 16);
+       assert(i >= 0 && i <= 0x10FFFF);
+       return(i);
 }
 
-
-/* 
- * Special character to string array.
- */
 const char *
-mchars_spec2str(struct mchars *arg, const char *p, size_t sz, size_t *rsz)
+mchars_spec2str(const struct mchars *arg,
+               const char *p, size_t sz, size_t *rsz)
 {
        const struct ln *ln;
 
-       ln = find(arg, p, sz, CHARS_CHAR);
-       if (NULL == ln)
-               return(NULL);
+       ln = find(arg, p, sz);
+       if (ln == NULL) {
+               *rsz = 1;
+               return(sz == 1 ? p : NULL);
+       }
 
        *rsz = strlen(ln->ascii);
        return(ln->ascii);
 }
 
-
-/* 
- * Reserved word to string array.
- */
 const char *
-mchars_res2str(struct mchars *arg, const char *p, size_t sz, size_t *rsz)
+mchars_uc2str(int uc)
 {
-       const struct ln *ln;
-
-       ln = find(arg, p, sz, CHARS_STRING);
-       if (NULL == ln)
-               return(NULL);
+       int      i;
 
-       *rsz = strlen(ln->ascii);
-       return(ln->ascii);
+       for (i = 0; i < LINES_MAX; i++)
+               if (uc == lines[i].unicode)
+                       return(lines[i].ascii);
+       return("<?>");
 }
 
 static const struct ln *
-find(struct mchars *tab, const char *p, size_t sz, int type)
+find(const struct mchars *tab, const char *p, size_t sz)
 {
-       struct ln        *pp, *prev;
-       struct ln       **htab;
+       const struct ln  *pp;
        int               hash;
 
        assert(p);
-       if (0 == sz)
-               return(NULL);
 
-       if (p[0] < PRINT_LO || p[0] > PRINT_HI)
+       if (0 == sz || p[0] < PRINT_LO || p[0] > PRINT_HI)
                return(NULL);
 
-       /*
-        * Lookup the symbol in the symbol hash.  See ascii2htab for the
-        * hashtable specs.  This dynamically re-orders the hash chain
-        * to optimise for repeat hits.
-        */
-
        hash = (int)p[0] - PRINT_LO;
-       htab = tab->htab;
-
-       if (NULL == (pp = htab[hash]))
-               return(NULL);
-
-       for (prev = NULL; pp; pp = pp->next) {
-               if ( ! match(pp, p, sz, type)) {
-                       prev = pp;
-                       continue;
-               }
 
-               if (prev) {
-                       prev->next = pp->next;
-                       pp->next = htab[hash];
-                       htab[hash] = pp;
-               }
-
-               return(pp);
-       }
+       for (pp = tab->htab[hash]; pp; pp = pp->next)
+               if (0 == strncmp(pp->code, p, sz) &&
+                   '\0' == pp->code[(int)sz])
+                       return(pp);
 
        return(NULL);
 }
-
-static inline int
-match(const struct ln *ln, const char *p, size_t sz, int type)
-{
-
-       if ( ! (ln->type & type))
-               return(0);
-       if (strncmp(ln->code, p, sz))
-               return(0);
-       return('\0' == ln->code[(int)sz]);
-}