ncurses 5.9 - patch 20120407
[ncurses.git] / progs / infocmp.c
index 0c36c2de03a74c4d67017795a5c1355fc81da4fb..2e6f67a8b6981e6648d2e36caf15f8925508d168 100644 (file)
@@ -1,5 +1,5 @@
 /****************************************************************************
 /****************************************************************************
- * Copyright (c) 1998-2008,2009 Free Software Foundation, Inc.              *
+ * Copyright (c) 1998-2011,2012 Free Software Foundation, Inc.              *
  *                                                                          *
  * Permission is hereby granted, free of charge, to any person obtaining a  *
  * copy of this software and associated documentation files (the            *
  *                                                                          *
  * Permission is hereby granted, free of charge, to any person obtaining a  *
  * copy of this software and associated documentation files (the            *
@@ -42,7 +42,7 @@
 
 #include <dump_entry.h>
 
 
 #include <dump_entry.h>
 
-MODULE_ID("$Id: infocmp.c,v 1.104 2009/12/05 21:10:31 tom Exp $")
+MODULE_ID("$Id: infocmp.c,v 1.115 2012/04/07 19:12:01 tom Exp $")
 
 #define L_CURL "{"
 #define R_CURL "}"
 
 #define L_CURL "{"
 #define R_CURL "}"
@@ -70,8 +70,9 @@ static const char *bool_sep = ":";
 static const char *s_absent = "NULL";
 static const char *s_cancel = "NULL";
 static const char *tversion;   /* terminfo version selected */
 static const char *s_absent = "NULL";
 static const char *s_cancel = "NULL";
 static const char *tversion;   /* terminfo version selected */
-static int itrace;             /* trace flag for debugging */
+static unsigned itrace;                /* trace flag for debugging */
 static int mwidth = 60;
 static int mwidth = 60;
+static int mheight = 65535;
 static int numbers = 0;                /* format "%'char'" to/from "%{number}" */
 static int outform = F_TERMINFO;       /* output format */
 static int sortmode;           /* sort_mode */
 static int numbers = 0;                /* format "%'char'" to/from "%{number}" */
 static int outform = F_TERMINFO;       /* output format */
 static int sortmode;           /* sort_mode */
@@ -106,7 +107,7 @@ canonical_name(char *ptr, char *buf)
 {
     char *bp;
 
 {
     char *bp;
 
-    (void) strcpy(buf, ptr);
+    _nc_STRCPY(buf, ptr, NAMESIZE);
     if ((bp = strchr(buf, '|')) != 0)
        *bp = '\0';
 
     if ((bp = strchr(buf, '|')) != 0)
        *bp = '\0';
 
@@ -307,13 +308,13 @@ dump_numeric(int val, char *buf)
 {
     switch (val) {
     case ABSENT_NUMERIC:
 {
     switch (val) {
     case ABSENT_NUMERIC:
-       strcpy(buf, s_absent);
+       _nc_STRCPY(buf, s_absent, MAX_STRING);
        break;
     case CANCELLED_NUMERIC:
        break;
     case CANCELLED_NUMERIC:
-       strcpy(buf, s_cancel);
+       _nc_STRCPY(buf, s_cancel, MAX_STRING);
        break;
     default:
        break;
     default:
-       sprintf(buf, "%d", val);
+       _nc_SPRINTF(buf, _nc_SLIMIT(MAX_STRING) "%d", val);
        break;
     }
 }
        break;
     }
 }
@@ -323,14 +324,30 @@ dump_string(char *val, char *buf)
 /* display the value of a string capability */
 {
     if (val == ABSENT_STRING)
 /* display the value of a string capability */
 {
     if (val == ABSENT_STRING)
-       strcpy(buf, s_absent);
+       _nc_STRCPY(buf, s_absent, MAX_STRING);
     else if (val == CANCELLED_STRING)
     else if (val == CANCELLED_STRING)
-       strcpy(buf, s_cancel);
+       _nc_STRCPY(buf, s_cancel, MAX_STRING);
     else {
     else {
-       sprintf(buf, "'%.*s'", MAX_STRING - 3, TIC_EXPAND(val));
+       _nc_SPRINTF(buf, _nc_SLIMIT(MAX_STRING)
+                   "'%.*s'", MAX_STRING - 3, TIC_EXPAND(val));
     }
 }
 
     }
 }
 
+/*
+ * ncurses stores two types of non-standard capabilities:
+ * a) capabilities listed past the "STOP-HERE" comment in the Caps file. 
+ *    These are used in the terminfo source file to provide data for termcaps,
+ *    e.g., when there is no equivalent capability in terminfo, as well as for
+ *    widely-used non-standard capabilities.
+ * b) user-definable capabilities, via "tic -x".
+ *
+ * However, if "-x" is omitted from the tic command, both types of
+ * non-standard capability are not loaded into the terminfo database.  This
+ * macro is used for limit-checks against the symbols that tic uses to omit
+ * the two types of non-standard entry.
+ */
+#define check_user_definable(n,limit) if (!_nc_user_definable && (n) > (limit)) break
+
 static void
 compare_predicate(PredType type, PredIdx idx, const char *name)
 /* predicate function to use for entry difference reports */
 static void
 compare_predicate(PredType type, PredIdx idx, const char *name)
 /* predicate function to use for entry difference reports */
@@ -344,6 +361,7 @@ compare_predicate(PredType type, PredIdx idx, const char *name)
 
     switch (type) {
     case CMP_BOOLEAN:
 
     switch (type) {
     case CMP_BOOLEAN:
+       check_user_definable(idx, BOOLWRITE);
        b1 = e1->tterm.Booleans[idx];
        b2 = e2->tterm.Booleans[idx];
        switch (compare) {
        b1 = e1->tterm.Booleans[idx];
        b2 = e2->tterm.Booleans[idx];
        switch (compare) {
@@ -369,6 +387,7 @@ compare_predicate(PredType type, PredIdx idx, const char *name)
        break;
 
     case CMP_NUMBER:
        break;
 
     case CMP_NUMBER:
+       check_user_definable(idx, NUMWRITE);
        n1 = e1->tterm.Numbers[idx];
        n2 = e2->tterm.Numbers[idx];
        dump_numeric(n1, buf1);
        n1 = e1->tterm.Numbers[idx];
        n2 = e2->tterm.Numbers[idx];
        dump_numeric(n1, buf1);
@@ -392,6 +411,7 @@ compare_predicate(PredType type, PredIdx idx, const char *name)
        break;
 
     case CMP_STRING:
        break;
 
     case CMP_STRING:
+       check_user_definable(idx, STRWRITE);
        s1 = e1->tterm.Strings[idx];
        s2 = e2->tterm.Strings[idx];
        switch (compare) {
        s1 = e1->tterm.Strings[idx];
        s2 = e2->tterm.Strings[idx];
        switch (compare) {
@@ -556,7 +576,7 @@ skip_csi(const char *cap)
 }
 
 static bool
 }
 
 static bool
-same_param(const char *table, const char *param, unsigned length)
+same_param(const char *table, const char *param, size_t length)
 {
     bool result = FALSE;
     if (strncmp(table, param, length) == 0) {
 {
     bool result = FALSE;
     if (strncmp(table, param, length) == 0) {
@@ -581,15 +601,15 @@ lookup_params(const assoc * table, char *dst, char *src)
                size_t tlen = strlen(ap->from);
 
                if (same_param(ap->from, ep, tlen)) {
                size_t tlen = strlen(ap->from);
 
                if (same_param(ap->from, ep, tlen)) {
-                   (void) strcat(dst, ap->to);
+                   _nc_STRCAT(dst, ap->to, MAX_TERMINFO_LENGTH);
                    found = TRUE;
                    break;
                }
            }
 
            if (!found)
                    found = TRUE;
                    break;
                }
            }
 
            if (!found)
-               (void) strcat(dst, ep);
-           (void) strcat(dst, ";");
+               _nc_STRCAT(dst, ep, MAX_TERMINFO_LENGTH);
+           _nc_STRCAT(dst, ";", MAX_TERMINFO_LENGTH);
        } while
            ((ep = strtok((char *) 0, ";")));
 
        } while
            ((ep = strtok((char *) 0, ";")));
 
@@ -682,7 +702,11 @@ analyze_string(const char *name, const char *cap, TERMTYPE *tp)
            && (next = (size_t) csi + len)
            && ((sp[next] == 'h') || (sp[next] == 'l'))) {
 
            && (next = (size_t) csi + len)
            && ((sp[next] == 'h') || (sp[next] == 'l'))) {
 
-           (void) strcpy(buf2, (sp[next] == 'h') ? "ECMA+" : "ECMA-");
+           _nc_STRCPY(buf2,
+                      ((sp[next] == 'h')
+                       ? "ECMA+"
+                       : "ECMA-"),
+                      sizeof(buf2));
            (void) strncpy(buf3, sp + csi, len);
            buf3[len] = '\0';
            len += (size_t) csi + 1;
            (void) strncpy(buf3, sp + csi, len);
            buf3[len] = '\0';
            len += (size_t) csi + 1;
@@ -699,7 +723,11 @@ analyze_string(const char *name, const char *cap, TERMTYPE *tp)
            && (next = (size_t) csi + 1 + len)
            && ((sp[next] == 'h') || (sp[next] == 'l'))) {
 
            && (next = (size_t) csi + 1 + len)
            && ((sp[next] == 'h') || (sp[next] == 'l'))) {
 
-           (void) strcpy(buf2, (sp[next] == 'h') ? "DEC+" : "DEC-");
+           _nc_STRCPY(buf2,
+                      ((sp[next] == 'h')
+                       ? "DEC+"
+                       : "DEC-"),
+                      sizeof(buf2));
            (void) strncpy(buf3, sp + csi + 1, len);
            buf3[len] = '\0';
            len += (size_t) csi + 2;
            (void) strncpy(buf3, sp + csi + 1, len);
            buf3[len] = '\0';
            len += (size_t) csi + 2;
@@ -715,7 +743,7 @@ analyze_string(const char *name, const char *cap, TERMTYPE *tp)
            && (next = (size_t) csi + len)
            && sp[next] == 'm') {
 
            && (next = (size_t) csi + len)
            && sp[next] == 'm') {
 
-           (void) strcpy(buf2, "SGR:");
+           _nc_STRCPY(buf2, "SGR:", sizeof(buf2));
            (void) strncpy(buf3, sp + csi, len);
            buf3[len] = '\0';
            len += (size_t) csi + 1;
            (void) strncpy(buf3, sp + csi, len);
            buf3[len] = '\0';
            len += (size_t) csi + 1;
@@ -727,8 +755,8 @@ analyze_string(const char *name, const char *cap, TERMTYPE *tp)
            && (csi = skip_csi(sp)) != 0
            && sp[csi] == 'm') {
            len = (size_t) csi + 1;
            && (csi = skip_csi(sp)) != 0
            && sp[csi] == 'm') {
            len = (size_t) csi + 1;
-           (void) strcpy(buf2, "SGR:");
-           strcat(buf2, ecma_highlights[0].to);
+           _nc_STRCPY(buf2, "SGR:", sizeof(buf2));
+           _nc_STRCAT(buf2, ecma_highlights[0].to, sizeof(buf2));
            expansion = buf2;
        }
 
            expansion = buf2;
        }
 
@@ -739,7 +767,7 @@ analyze_string(const char *name, const char *cap, TERMTYPE *tp)
                expansion = "RSR";
                len = 1;
            } else {
                expansion = "RSR";
                len = 1;
            } else {
-               (void) sprintf(buf2, "1;%dr", tp_lines);
+               _nc_SPRINTF(buf2, _nc_SLIMIT(sizeof(buf2)) "1;%dr", tp_lines);
                len = strlen(buf2);
                if (strncmp(buf2, sp + csi, len) == 0)
                    expansion = "RSR";
                len = strlen(buf2);
                if (strncmp(buf2, sp + csi, len) == 0)
                    expansion = "RSR";
@@ -750,12 +778,12 @@ analyze_string(const char *name, const char *cap, TERMTYPE *tp)
        /* now check for home-down */
        if (!expansion
            && (csi = skip_csi(sp)) != 0) {
        /* now check for home-down */
        if (!expansion
            && (csi = skip_csi(sp)) != 0) {
-           (void) sprintf(buf2, "%d;1H", tp_lines);
+           _nc_SPRINTF(buf2, _nc_SLIMIT(sizeof(buf2)) "%d;1H", tp_lines);
            len = strlen(buf2);
            if (strncmp(buf2, sp + csi, len) == 0) {
                expansion = "LL";
            } else {
            len = strlen(buf2);
            if (strncmp(buf2, sp + csi, len) == 0) {
                expansion = "LL";
            } else {
-               (void) sprintf(buf2, "%dH", tp_lines);
+               _nc_SPRINTF(buf2, _nc_SLIMIT(sizeof(buf2)) "%dH", tp_lines);
                len = strlen(buf2);
                if (strncmp(buf2, sp + csi, len) == 0) {
                    expansion = "LL";
                len = strlen(buf2);
                if (strncmp(buf2, sp + csi, len) == 0) {
                    expansion = "LL";
@@ -795,7 +823,7 @@ file_comparison(int argc, char *argv[])
     int i, n;
 
     memset(heads, 0, sizeof(heads));
     int i, n;
 
     memset(heads, 0, sizeof(heads));
-    dump_init((char *) 0, F_LITERAL, S_TERMINFO, 0, itrace, FALSE);
+    dump_init((char *) 0, F_LITERAL, S_TERMINFO, 0, 65535, itrace, FALSE);
 
     for (n = 0; n < argc && n < MAXCOMPARE; n++) {
        if (freopen(argv[n], "r", stdin) == 0)
 
     for (n = 0; n < argc && n < MAXCOMPARE; n++) {
        if (freopen(argv[n], "r", stdin) == 0)
@@ -894,8 +922,6 @@ file_comparison(int argc, char *argv[])
 
     (void) printf("The following entries are equivalent:\n");
     for (qp = heads[0]; qp; qp = qp->next) {
 
     (void) printf("The following entries are equivalent:\n");
     for (qp = heads[0]; qp; qp = qp->next) {
-       rp = qp->crosslinks[0];
-
        if (qp->ncrosslinks == 1) {
            rp = qp->crosslinks[0];
 
        if (qp->ncrosslinks == 1) {
            rp = qp->crosslinks[0];
 
@@ -976,7 +1002,9 @@ usage(void)
        "Usage: infocmp [options] [-A directory] [-B directory] [termname...]"
        ,""
        ,"Options:"
        "Usage: infocmp [options] [-A directory] [-B directory] [termname...]"
        ,""
        ,"Options:"
+       ,"  -0    print single-row"
        ,"  -1    print single-column"
        ,"  -1    print single-column"
+       ,"  -K    use termcap-names and BSD syntax"
        ,"  -C    use termcap-names"
        ,"  -F    compare terminfo-files"
        ,"  -I    use terminfo-names"
        ,"  -C    use termcap-names"
        ,"  -F    compare terminfo-files"
        ,"  -I    use terminfo-names"
@@ -1032,19 +1060,23 @@ static char *
 any_initializer(const char *fmt, const char *type)
 {
     static char *initializer;
 any_initializer(const char *fmt, const char *type)
 {
     static char *initializer;
+    static size_t need;
     char *s;
 
     char *s;
 
-    if (initializer == 0)
-       initializer = (char *) malloc(strlen(entries->tterm.term_names) +
-                                     strlen(type) + strlen(fmt));
+    if (initializer == 0) {
+       need = (strlen(entries->tterm.term_names)
+               + strlen(type)
+               + strlen(fmt));
+       initializer = (char *) malloc(need);
+    }
 
 
-    (void) strcpy(initializer, entries->tterm.term_names);
+    _nc_STRCPY(initializer, entries->tterm.term_names, need);
     for (s = initializer; *s != 0 && *s != '|'; s++) {
        if (!isalnum(UChar(*s)))
            *s = '_';
     }
     *s = 0;
     for (s = initializer; *s != 0 && *s != '|'; s++) {
        if (!isalnum(UChar(*s)))
            *s = '_';
     }
     *s = 0;
-    (void) sprintf(s, fmt, type);
+    _nc_SPRINTF(s, _nc_SLIMIT(need) fmt, type);
     return initializer;
 }
 
     return initializer;
 }
 
@@ -1075,9 +1107,10 @@ dump_initializers(TERMTYPE *term)
 
        if (VALID_STRING(term->Strings[n])) {
            tp = buf;
 
        if (VALID_STRING(term->Strings[n])) {
            tp = buf;
+#define TP_LIMIT       ((MAX_STRING - 5) - (size_t)(tp - buf))
            *tp++ = '"';
            for (sp = term->Strings[n];
            *tp++ = '"';
            for (sp = term->Strings[n];
-                *sp != 0 && (tp - buf) < MAX_STRING - 6;
+                *sp != 0 && TP_LIMIT > 2;
                 sp++) {
                if (isascii(UChar(*sp))
                    && isprint(UChar(*sp))
                 sp++) {
                if (isascii(UChar(*sp))
                    && isprint(UChar(*sp))
@@ -1085,14 +1118,15 @@ dump_initializers(TERMTYPE *term)
                    && *sp != '"')
                    *tp++ = *sp;
                else {
                    && *sp != '"')
                    *tp++ = *sp;
                else {
-                   (void) sprintf(tp, "\\%03o", UChar(*sp));
+                   _nc_SPRINTF(tp, _nc_SLIMIT(TP_LIMIT) "\\%03o", UChar(*sp));
                    tp += 4;
                }
            }
            *tp++ = '"';
            *tp = '\0';
            (void) printf("static char %-20s[] = %s;\n",
                    tp += 4;
                }
            }
            *tp++ = '"';
            *tp = '\0';
            (void) printf("static char %-20s[] = %s;\n",
-                         string_variable(ExtStrname(term, n, strnames)), buf);
+                         string_variable(ExtStrname(term, (int) n, strnames)),
+                         buf);
        }
     }
     printf("\n");
        }
     }
     printf("\n");
@@ -1118,7 +1152,7 @@ dump_initializers(TERMTYPE *term)
            break;
        }
        (void) printf("\t/* %3u: %-8s */\t%s,\n",
            break;
        }
        (void) printf("\t/* %3u: %-8s */\t%s,\n",
-                     n, ExtBoolname(term, n, boolnames), str);
+                     n, ExtBoolname(term, (int) n, boolnames), str);
     }
     (void) printf("%s;\n", R_CURL);
 
     }
     (void) printf("%s;\n", R_CURL);
 
@@ -1134,12 +1168,12 @@ dump_initializers(TERMTYPE *term)
            str = "CANCELLED_NUMERIC";
            break;
        default:
            str = "CANCELLED_NUMERIC";
            break;
        default:
-           sprintf(buf, "%d", term->Numbers[n]);
+           _nc_SPRINTF(buf, _nc_SLIMIT(sizeof(buf)) "%d", term->Numbers[n]);
            str = buf;
            break;
        }
        (void) printf("\t/* %3u: %-8s */\t%s,\n", n,
            str = buf;
            break;
        }
        (void) printf("\t/* %3u: %-8s */\t%s,\n", n,
-                     ExtNumname(term, n, numnames), str);
+                     ExtNumname(term, (int) n, numnames), str);
     }
     (void) printf("%s;\n", R_CURL);
 
     }
     (void) printf("%s;\n", R_CURL);
 
@@ -1152,10 +1186,10 @@ dump_initializers(TERMTYPE *term)
        else if (term->Strings[n] == CANCELLED_STRING)
            str = "CANCELLED_STRING";
        else {
        else if (term->Strings[n] == CANCELLED_STRING)
            str = "CANCELLED_STRING";
        else {
-           str = string_variable(ExtStrname(term, n, strnames));
+           str = string_variable(ExtStrname(term, (int) n, strnames));
        }
        (void) printf("\t/* %3u: %-8s */\t%s,\n", n,
        }
        (void) printf("\t/* %3u: %-8s */\t%s,\n", n,
-                     ExtStrname(term, n, strnames), str);
+                     ExtStrname(term, (int) n, strnames), str);
     }
     (void) printf("%s;\n", R_CURL);
 
     }
     (void) printf("%s;\n", R_CURL);
 
@@ -1167,15 +1201,15 @@ dump_initializers(TERMTYPE *term)
                      name_initializer("string_ext"), L_CURL);
        for (n = BOOLCOUNT; n < NUM_BOOLEANS(term); ++n) {
            (void) printf("\t/* %3u: bool */\t\"%s\",\n",
                      name_initializer("string_ext"), L_CURL);
        for (n = BOOLCOUNT; n < NUM_BOOLEANS(term); ++n) {
            (void) printf("\t/* %3u: bool */\t\"%s\",\n",
-                         n, ExtBoolname(term, n, boolnames));
+                         n, ExtBoolname(term, (int) n, boolnames));
        }
        for (n = NUMCOUNT; n < NUM_NUMBERS(term); ++n) {
            (void) printf("\t/* %3u: num */\t\"%s\",\n",
        }
        for (n = NUMCOUNT; n < NUM_NUMBERS(term); ++n) {
            (void) printf("\t/* %3u: num */\t\"%s\",\n",
-                         n, ExtNumname(term, n, numnames));
+                         n, ExtNumname(term, (int) n, numnames));
        }
        for (n = STRCOUNT; n < NUM_STRINGS(term); ++n) {
            (void) printf("\t/* %3u: str */\t\"%s\",\n",
        }
        for (n = STRCOUNT; n < NUM_STRINGS(term); ++n) {
            (void) printf("\t/* %3u: str */\t\"%s\",\n",
-                         n, ExtStrname(term, n, strnames));
+                         n, ExtStrname(term, (int) n, strnames));
        }
        (void) printf("%s;\n", R_CURL);
     }
        }
        (void) printf("%s;\n", R_CURL);
     }
@@ -1249,6 +1283,23 @@ terminal_env(void)
     return terminal;
 }
 
     return terminal;
 }
 
+/*
+ * Show the databases that infocmp knows about.  The location to which it writes is
+ */
+static void
+show_databases(void)
+{
+    DBDIRS state;
+    int offset;
+    const char *path2;
+
+    _nc_first_db(&state, &offset);
+    while ((path2 = _nc_next_db(&state, &offset)) != 0) {
+       printf("%s\n", path2);
+    }
+    _nc_last_db();
+}
+
 /***************************************************************************
  *
  * Main sequence
 /***************************************************************************
  *
  * Main sequence
@@ -1271,7 +1322,7 @@ main(int argc, char *argv[])
     /* Also avoid overflowing smaller stacks on systems like AmigaOS */
     path *tfile = 0;
     char **tname = 0;
     /* Also avoid overflowing smaller stacks on systems like AmigaOS */
     path *tfile = 0;
     char **tname = 0;
-    int maxterms;
+    size_t maxterms;
 
     char **myargv;
 
 
     char **myargv;
 
@@ -1289,6 +1340,7 @@ main(int argc, char *argv[])
 #if NCURSES_XNAMES
     use_extended_names(FALSE);
 #endif
 #if NCURSES_XNAMES
     use_extended_names(FALSE);
 #endif
+    _nc_strict_bsd = 0;
 
     _nc_progname = _nc_rootname(argv[0]);
 
 
     _nc_progname = _nc_rootname(argv[0]);
 
@@ -1299,8 +1351,13 @@ main(int argc, char *argv[])
 
     while ((c = getopt(argc,
                       argv,
 
     while ((c = getopt(argc,
                       argv,
-                      "1A:aB:CcdEeFfGgIiLlnpqR:rs:TtUuVv:w:x")) != -1) {
+                      "01A:aB:CcDdEeFfGgIiKLlnpqR:rs:TtUuVv:w:x")) != -1) {
        switch (c) {
        switch (c) {
+       case '0':
+           mwidth = 65535;
+           mheight = 1;
+           break;
+
        case '1':
            mwidth = 0;
            break;
        case '1':
            mwidth = 0;
            break;
@@ -1319,6 +1376,9 @@ main(int argc, char *argv[])
            restdir = optarg;
            break;
 
            restdir = optarg;
            break;
 
+       case 'K':
+           _nc_strict_bsd = 1;
+           /* FALLTHRU */
        case 'C':
            outform = F_TERMCAP;
            tversion = "BSD";
        case 'C':
            outform = F_TERMCAP;
            tversion = "BSD";
@@ -1326,6 +1386,11 @@ main(int argc, char *argv[])
                sortmode = S_TERMCAP;
            break;
 
                sortmode = S_TERMCAP;
            break;
 
+       case 'D':
+           show_databases();
+           ExitProgram(EXIT_SUCCESS);
+           break;
+
        case 'c':
            compare = C_COMMON;
            break;
        case 'c':
            compare = C_COMMON;
            break;
@@ -1443,7 +1508,7 @@ main(int argc, char *argv[])
            ExitProgram(EXIT_SUCCESS);
 
        case 'v':
            ExitProgram(EXIT_SUCCESS);
 
        case 'v':
-           itrace = optarg_to_number();
+           itrace = (unsigned) optarg_to_number();
            set_trace_level(itrace);
            break;
 
            set_trace_level(itrace);
            break;
 
@@ -1462,7 +1527,7 @@ main(int argc, char *argv[])
        }
     }
 
        }
     }
 
-    maxterms = (argc + 2 - optind);
+    maxterms = (size_t) (argc + 2 - optind);
     tfile = typeMalloc(path, maxterms);
     tname = typeCalloc(char *, maxterms);
     entries = typeCalloc(ENTRY, maxterms);
     tfile = typeMalloc(path, maxterms);
     tname = typeCalloc(char *, maxterms);
     entries = typeCalloc(ENTRY, maxterms);
@@ -1479,7 +1544,7 @@ main(int argc, char *argv[])
        sortmode = S_TERMINFO;
 
     /* set up for display */
        sortmode = S_TERMINFO;
 
     /* set up for display */
-    dump_init(tversion, outform, sortmode, mwidth, itrace, formatted);
+    dump_init(tversion, outform, sortmode, mwidth, mheight, itrace, formatted);
 
     /* make sure we have at least one terminal name to work with */
     if (optind >= argc)
 
     /* make sure we have at least one terminal name to work with */
     if (optind >= argc)
@@ -1509,9 +1574,11 @@ main(int argc, char *argv[])
 #else
 #define LEAF_FMT "%02x"
 #endif
 #else
 #define LEAF_FMT "%02x"
 #endif
-               (void) sprintf(tfile[termcount], "%s/" LEAF_FMT "/%s",
-                              directory,
-                              UChar(*argv[optind]), argv[optind]);
+               _nc_SPRINTF(tfile[termcount],
+                           _nc_SLIMIT(sizeof(path))
+                           "%s/" LEAF_FMT "/%s",
+                           directory,
+                           UChar(*argv[optind]), argv[optind]);
                if (itrace)
                    (void) fprintf(stderr,
                                   "%s: reading entry %s from file %s\n",
                if (itrace)
                    (void) fprintf(stderr,
                                   "%s: reading entry %s from file %s\n",
@@ -1536,7 +1603,6 @@ main(int argc, char *argv[])
                status = _nc_read_entry(tname[termcount],
                                        tfile[termcount],
                                        &entries[termcount].tterm);
                status = _nc_read_entry(tname[termcount],
                                        tfile[termcount],
                                        &entries[termcount].tterm);
-               directory = TERMINFO;   /* for error message */
            }
 
            if (status <= 0) {
            }
 
            if (status <= 0) {