Do not let the -m option or MANPATH with leading, trailing, or double

colon override the default manpath, let them add to the default manpath.
Only override the default manpath by the -M option, by MANPATH without
leading, trailing, or double colon, or by "manpath" in man.conf(5).

Problem reported by Jan Stary <hans at stare dot cz>.
Patch OK'ed by millert@.

diff --git a/manpath.c b/manpath.c
index 2bb83684b6e8d878508dff80c08c9991415feac4..9446ea6971a42cc81e477b6161e107ae02215848 100644 (file)
--- a/manpath.c
+++ b/manpath.c
@@ -1,4 +1,4 @@
-/*     $Id: manpath.c,v 1.24 2015/04/28 16:13:45 schwarze Exp $        */
+/*     $Id: manpath.c,v 1.25 2015/05/07 12:08:13 schwarze Exp $        */

 /*
  * Copyright (c) 2011, 2014, 2015 Ingo Schwarze <schwarze@openbsd.org>
  * Copyright (c) 2011 Kristaps Dzonsons <kristaps@bsd.lv>
 /*
  * Copyright (c) 2011, 2014, 2015 Ingo Schwarze <schwarze@openbsd.org>
  * Copyright (c) 2011 Kristaps Dzonsons <kristaps@bsd.lv>


@@ -84,7 +84,6 @@ manconf_parse(struct manconf *conf, const char *file,
        free(buf);
        pclose(stream);
 #else
        free(buf);
        pclose(stream);
 #else

-       char             manpath_default[] = MANPATH_DEFAULT;

        char            *insert;
 
        /* Always prepend -m. */
        char            *insert;
 
        /* Always prepend -m. */


@@ -104,8 +103,6 @@ manconf_parse(struct manconf *conf, const char *file,
        /* No MANPATH; use man.conf(5) only. */
        if (NULL == defp || '\0' == defp[0]) {
                manconf_file(conf, file);
        /* No MANPATH; use man.conf(5) only. */
        if (NULL == defp || '\0' == defp[0]) {
                manconf_file(conf, file);

-               if (conf->manpath.sz == 0)
-                       manpath_parseline(&conf->manpath, manpath_default, 0);

                return;
        }
 
                return;
        }
 


@@ -210,13 +207,14 @@ static void
 manconf_file(struct manconf *conf, const char *file)
 {
        const char *const toks[] = { "manpath", "output", "_whatdb" };
 manconf_file(struct manconf *conf, const char *file)
 {
        const char *const toks[] = { "manpath", "output", "_whatdb" };

+       char manpath_default[] = MANPATH_DEFAULT;

 
        FILE            *stream;
        char            *cp, *ep;
        size_t           len, tok;
 
        if ((stream = fopen(file, "r")) == NULL)
 
        FILE            *stream;
        char            *cp, *ep;
        size_t           len, tok;
 
        if ((stream = fopen(file, "r")) == NULL)

-               return;
+               goto out;

 
        while ((cp = fgetln(stream, &len)) != NULL) {
                ep = cp + len;
 
        while ((cp = fgetln(stream, &len)) != NULL) {
                ep = cp + len;


@@ -250,6 +248,7 @@ manconf_file(struct manconf *conf, const char *file)
                        /* FALLTHROUGH */
                case 0:  /* manpath */
                        manpath_add(&conf->manpath, cp, 0);
                        /* FALLTHROUGH */
                case 0:  /* manpath */
                        manpath_add(&conf->manpath, cp, 0);

+                       *manpath_default = '\0';

                        break;
                case 1:  /* output */
                        manconf_output(&conf->output, cp);
                        break;
                case 1:  /* output */
                        manconf_output(&conf->output, cp);


@@ -258,8 +257,11 @@ manconf_file(struct manconf *conf, const char *file)
                        break;
                }
        }
                        break;
                }
        }

-

        fclose(stream);
        fclose(stream);

+
+out:
+       if (*manpath_default != '\0')
+               manpath_parseline(&conf->manpath, manpath_default, 0);

 }
 
 void
 }
 
 void