Sevan Janiyan resigned from the NetBSD project and pkgsrc in 2020.

[mandoc.git] / manpath.c

diff --git a/manpath.c b/manpath.c
index f43ace60694ea3a6126fa4b4160d4b1b778c3ff1..255d748246d61e36a4e3e28dce4adb68ee16b7af 100644 (file)
--- a/manpath.c
+++ b/manpath.c
@@ -1,6 +1,6 @@
-/*     $Id: manpath.c,v 1.33 2017/02/10 15:45:28 schwarze Exp $        */
+/*     $Id: manpath.c,v 1.43 2020/08/27 14:59:47 schwarze Exp $ */

 /*
 /*

- * Copyright (c) 2011, 2014, 2015, 2017 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2011,2014,2015,2017-2019 Ingo Schwarze <schwarze@openbsd.org>

  * Copyright (c) 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * Copyright (c) 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  *
  * Permission to use, copy, modify, and distribute this software for any


@@ -21,20 +21,19 @@
 #include <sys/stat.h>
 
 #include <ctype.h>
 #include <sys/stat.h>
 
 #include <ctype.h>

-#if HAVE_ERR
-#include <err.h>
-#endif
+#include <errno.h>

 #include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
 #include "mandoc_aux.h"
 #include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
 #include "mandoc_aux.h"

+#include "mandoc.h"

 #include "manconf.h"
 
 static void     manconf_file(struct manconf *, const char *);
 #include "manconf.h"
 
 static void     manconf_file(struct manconf *, const char *);

-static void     manpath_add(struct manpaths *, const char *, int);
-static void     manpath_parseline(struct manpaths *, char *, int);
+static void     manpath_add(struct manpaths *, const char *, char);
+static void     manpath_parseline(struct manpaths *, char *, char);

 
 
 void
 
 
 void


@@ -44,11 +43,11 @@ manconf_parse(struct manconf *conf, const char *file,
        char            *insert;
 
        /* Always prepend -m. */
        char            *insert;
 
        /* Always prepend -m. */

-       manpath_parseline(&conf->manpath, auxp, 1);
+       manpath_parseline(&conf->manpath, auxp, 'm');

 
        /* If -M is given, it overrides everything else. */
        if (NULL != defp) {
 
        /* If -M is given, it overrides everything else. */
        if (NULL != defp) {

-               manpath_parseline(&conf->manpath, defp, 1);
+               manpath_parseline(&conf->manpath, defp, 'M');

                return;
        }
 
                return;
        }
 


@@ -66,13 +65,13 @@ manconf_parse(struct manconf *conf, const char *file,
        /* Prepend man.conf(5) to MANPATH. */
        if (':' == defp[0]) {
                manconf_file(conf, file);
        /* Prepend man.conf(5) to MANPATH. */
        if (':' == defp[0]) {
                manconf_file(conf, file);

-               manpath_parseline(&conf->manpath, defp, 0);
+               manpath_parseline(&conf->manpath, defp, '\0');

                return;
        }
 
        /* Append man.conf(5) to MANPATH. */
        if (':' == defp[strlen(defp) - 1]) {
                return;
        }
 
        /* Append man.conf(5) to MANPATH. */
        if (':' == defp[strlen(defp) - 1]) {

-               manpath_parseline(&conf->manpath, defp, 0);
+               manpath_parseline(&conf->manpath, defp, '\0');

                manconf_file(conf, file);
                return;
        }
                manconf_file(conf, file);
                return;
        }


@@ -81,21 +80,28 @@ manconf_parse(struct manconf *conf, const char *file,
        insert = strstr(defp, "::");
        if (NULL != insert) {
                *insert++ = '\0';
        insert = strstr(defp, "::");
        if (NULL != insert) {
                *insert++ = '\0';

-               manpath_parseline(&conf->manpath, defp, 0);
+               manpath_parseline(&conf->manpath, defp, '\0');

                manconf_file(conf, file);
                manconf_file(conf, file);

-               manpath_parseline(&conf->manpath, insert + 1, 0);
+               manpath_parseline(&conf->manpath, insert + 1, '\0');

                return;
        }
 
        /* MANPATH overrides man.conf(5) completely. */
                return;
        }
 
        /* MANPATH overrides man.conf(5) completely. */

-       manpath_parseline(&conf->manpath, defp, 0);
+       manpath_parseline(&conf->manpath, defp, '\0');
+}
+
+void
+manpath_base(struct manpaths *dirs)
+{
+       char path_base[] = MANPATH_BASE;
+       manpath_parseline(dirs, path_base, '\0');

 }
 
 /*
  * Parse a FULL pathname from a colon-separated list of arrays.
  */
 static void
 }
 
 /*
  * Parse a FULL pathname from a colon-separated list of arrays.
  */
 static void

-manpath_parseline(struct manpaths *dirs, char *path, int complain)
+manpath_parseline(struct manpaths *dirs, char *path, char option)

 {
        char    *dir;
 
 {
        char    *dir;
 


@@ -103,7 +109,7 @@ manpath_parseline(struct manpaths *dirs, char *path, int complain)
                return;
 
        for (dir = strtok(path, ":"); dir; dir = strtok(NULL, ":"))
                return;
 
        for (dir = strtok(path, ":"); dir; dir = strtok(NULL, ":"))

-               manpath_add(dirs, dir, complain);
+               manpath_add(dirs, dir, option);

 }
 
 /*
 }
 
 /*


@@ -111,33 +117,32 @@ manpath_parseline(struct manpaths *dirs, char *path, int complain)
  * Grow the array one-by-one for simplicity's sake.
  */
 static void
  * Grow the array one-by-one for simplicity's sake.
  */
 static void

-manpath_add(struct manpaths *dirs, const char *dir, int complain)
+manpath_add(struct manpaths *dirs, const char *dir, char option)

 {
        char             buf[PATH_MAX];
        struct stat      sb;
        char            *cp;
        size_t           i;
 
 {
        char             buf[PATH_MAX];
        struct stat      sb;
        char            *cp;
        size_t           i;
 

-       if (NULL == (cp = realpath(dir, buf))) {
-               if (complain)
-                       warn("manpath: %s", dir);
-               return;
-       }
+       if ((cp = realpath(dir, buf)) == NULL)
+               goto fail;

 
        for (i = 0; i < dirs->sz; i++)
 
        for (i = 0; i < dirs->sz; i++)

-               if (0 == strcmp(dirs->paths[i], dir))
+               if (strcmp(dirs->paths[i], dir) == 0)

                        return;
 
                        return;
 

-       if (stat(cp, &sb) == -1) {
-               if (complain)
-                       warn("manpath: %s", dir);
-               return;
-       }
+       if (stat(cp, &sb) == -1)
+               goto fail;

 
        dirs->paths = mandoc_reallocarray(dirs->paths,
 
        dirs->paths = mandoc_reallocarray(dirs->paths,

-           dirs->sz + 1, sizeof(char *));
-
+           dirs->sz + 1, sizeof(*dirs->paths));

        dirs->paths[dirs->sz++] = mandoc_strdup(cp);
        dirs->paths[dirs->sz++] = mandoc_strdup(cp);

+       return;
+
+fail:
+       if (option != '\0')
+               mandoc_msg(MANDOCERR_BADARG_BAD, 0, 0,
+                   "-%c %s: %s", option, dir, strerror(errno));

 }
 
 void
 }
 
 void


@@ -158,7 +163,7 @@ manconf_free(struct manconf *conf)
 static void
 manconf_file(struct manconf *conf, const char *file)
 {
 static void
 manconf_file(struct manconf *conf, const char *file)
 {

-       const char *const toks[] = { "manpath", "output", "_whatdb" };
+       const char *const toks[] = { "manpath", "output" };

        char manpath_default[] = MANPATH_DEFAULT;
 
        FILE            *stream;
        char manpath_default[] = MANPATH_DEFAULT;
 
        FILE            *stream;


@@ -195,15 +200,8 @@ manconf_file(struct manconf *conf, const char *file)
                }
 
                switch (tok) {
                }
 
                switch (tok) {

-               case 2:  /* _whatdb */
-                       while (ep > cp && ep[-1] != '/')
-                               ep--;
-                       if (ep == cp)
-                               continue;
-                       *ep = '\0';
-                       /* FALLTHROUGH */

                case 0:  /* manpath */
                case 0:  /* manpath */

-                       manpath_add(&conf->manpath, cp, 0);
+                       manpath_add(&conf->manpath, cp, '\0');

                        *manpath_default = '\0';
                        break;
                case 1:  /* output */
                        *manpath_default = '\0';
                        break;
                case 1:  /* output */


@@ -218,24 +216,30 @@ manconf_file(struct manconf *conf, const char *file)
 
 out:
        if (*manpath_default != '\0')
 
 out:
        if (*manpath_default != '\0')

-               manpath_parseline(&conf->manpath, manpath_default, 0);
+               manpath_parseline(&conf->manpath, manpath_default, '\0');

 }
 
 int
 manconf_output(struct manoutput *conf, const char *cp, int fromfile)
 {
        const char *const toks[] = {
 }
 
 int
 manconf_output(struct manoutput *conf, const char *cp, int fromfile)
 {
        const char *const toks[] = {

-           "includes", "man", "paper", "style",
-           "indent", "width", "fragment", "mdoc", "noval"
+           /* Tokens requiring an argument. */
+           "includes", "man", "paper", "style", "indent", "width",
+           "outfilename", "tagfilename",
+           /* Token taking an optional argument. */
+           "tag",
+           /* Tokens not taking arguments. */
+           "fragment", "mdoc", "noval", "toc"

        };
        };

+       const size_t ntoks = sizeof(toks) / sizeof(toks[0]);

 
        const char      *errstr;
        char            *oldval;
        size_t           len, tok;
 
 
        const char      *errstr;
        char            *oldval;
        size_t           len, tok;
 

-       for (tok = 0; tok < sizeof(toks)/sizeof(toks[0]); tok++) {
+       for (tok = 0; tok < ntoks; tok++) {

                len = strlen(toks[tok]);
                len = strlen(toks[tok]);

-               if ( ! strncmp(cp, toks[tok], len) &&
+               if (strncmp(cp, toks[tok], len) == 0 &&

                    strchr(" =  ", cp[len]) != NULL) {
                        cp += len;
                        if (*cp == '=')
                    strchr(" =  ", cp[len]) != NULL) {
                        cp += len;
                        if (*cp == '=')


@@ -246,12 +250,12 @@ manconf_output(struct manoutput *conf, const char *cp, int fromfile)
                }
        }
 
                }
        }
 

-       if (tok < 6 && *cp == '\0') {
-               warnx("-O %s=?: Missing argument value", toks[tok]);
+       if (tok < 8 && *cp == '\0') {
+               mandoc_msg(MANDOCERR_BADVAL_MISS, 0, 0, "-O %s=?", toks[tok]);

                return -1;
        }
                return -1;
        }

-       if ((tok == 6 || tok == 7) && *cp != '\0') {
-               warnx("-O %s: Does not take a value: %s", toks[tok], cp);
+       if (tok > 8 && tok < ntoks && *cp != '\0') {
+               mandoc_msg(MANDOCERR_BADVAL, 0, 0, "-O %s=%s", toks[tok], cp);

                return -1;
        }
 
                return -1;
        }
 


@@ -292,35 +296,68 @@ manconf_output(struct manoutput *conf, const char *cp, int fromfile)
                conf->indent = strtonum(cp, 0, 1000, &errstr);
                if (errstr == NULL)
                        return 0;
                conf->indent = strtonum(cp, 0, 1000, &errstr);
                if (errstr == NULL)
                        return 0;

-               warnx("-O indent=%s is %s", cp, errstr);
+               mandoc_msg(MANDOCERR_BADVAL_BAD, 0, 0,
+                   "-O indent=%s is %s", cp, errstr);

                return -1;
        case 5:
                if (conf->width) {
                        mandoc_asprintf(&oldval, "%zu", conf->width);
                        break;
                }
                return -1;
        case 5:
                if (conf->width) {
                        mandoc_asprintf(&oldval, "%zu", conf->width);
                        break;
                }

-               conf->width = strtonum(cp, 58, 1000, &errstr);
+               conf->width = strtonum(cp, 1, 1000, &errstr);

                if (errstr == NULL)
                        return 0;
                if (errstr == NULL)
                        return 0;

-               warnx("-O width=%s is %s", cp, errstr);
+               mandoc_msg(MANDOCERR_BADVAL_BAD, 0, 0,
+                   "-O width=%s is %s", cp, errstr);

                return -1;
        case 6:
                return -1;
        case 6:

-               conf->fragment = 1;
+               if (conf->outfilename != NULL) {
+                       oldval = mandoc_strdup(conf->outfilename);
+                       break;
+               }
+               conf->outfilename = mandoc_strdup(cp);

                return 0;
        case 7:
                return 0;
        case 7:

-               conf->mdoc = 1;
+               if (conf->tagfilename != NULL) {
+                       oldval = mandoc_strdup(conf->tagfilename);
+                       break;
+               }
+               conf->tagfilename = mandoc_strdup(cp);

                return 0;
                return 0;

+       /*
+        * If the index of the following token changes,
+        * do not forget to adjust the range check above the switch.
+        */

        case 8:
        case 8:

+               if (conf->tag != NULL) {
+                       oldval = mandoc_strdup(conf->tag);
+                       break;
+               }
+               conf->tag = mandoc_strdup(cp);
+               return 0;
+       case 9:
+               conf->fragment = 1;
+               return 0;
+       case 10:
+               conf->mdoc = 1;
+               return 0;
+       case 11:

                conf->noval = 1;
                return 0;
                conf->noval = 1;
                return 0;

+       case 12:
+               conf->toc = 1;
+               return 0;

        default:
        default:

-               if (fromfile)
-                       warnx("-O %s: Bad argument", cp);
+               mandoc_msg(MANDOCERR_BADARG_BAD, 0, 0, "-O %s", cp);
+               return -1;
+       }
+       if (fromfile) {
+               free(oldval);
+               return 0;
+       } else {
+               mandoc_msg(MANDOCERR_BADVAL_DUPE, 0, 0,
+                   "-O %s=%s: already set to %s", toks[tok], cp, oldval);
+               free(oldval);

                return -1;
        }
                return -1;
        }

-       if (fromfile == 0)
-               warnx("-O %s=%s: Option already set to %s",
-                   toks[tok], cp, oldval);
-       free(oldval);
-       return -1;

 }
 }