On a new RS nesting level, the saved width starts from the default

[mandoc.git] / read.c

diff --git a/read.c b/read.c
index cad96e2de037052fc3321f40fe7ca6d59d1f72c2..8adad316e96db488059fc1e0575d98ec19346b53 100644 (file)
--- a/read.c
+++ b/read.c
@@ -1,16 +1,16 @@
-/*     $Id: read.c,v 1.97 2014/11/26 21:40:17 schwarze Exp $ */
+/*     $Id: read.c,v 1.133 2015/04/02 21:36:50 schwarze Exp $ */

 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>

- * Copyright (c) 2010-2014 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2010-2015 Ingo Schwarze <schwarze@openbsd.org>

  * Copyright (c) 2010, 2012 Joerg Sonnenberger <joerg@netbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
  * copyright notice and this permission notice appear in all copies.
  *
  * Copyright (c) 2010, 2012 Joerg Sonnenberger <joerg@netbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
  * copyright notice and this permission notice appear in all copies.
  *

- * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHORS DISCLAIM ALL WARRANTIES

  * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
  * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF

- * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR

  * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
  * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
  * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF


@@ -29,6 +29,7 @@
 #include <ctype.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <ctype.h>
 #include <errno.h>
 #include <fcntl.h>

+#include <signal.h>

 #include <stdarg.h>
 #include <stdint.h>
 #include <stdio.h>
 #include <stdarg.h>
 #include <stdint.h>
 #include <stdio.h>


@@ -36,12 +37,12 @@
 #include <string.h>
 #include <unistd.h>
 
 #include <string.h>
 #include <unistd.h>
 

-#include "mandoc.h"

 #include "mandoc_aux.h"
 #include "mandoc_aux.h"

-#include "libmandoc.h"
+#include "mandoc.h"
+#include "roff.h"

 #include "mdoc.h"
 #include "man.h"
 #include "mdoc.h"
 #include "man.h"

-#include "main.h"
+#include "libmandoc.h"

 
 #define        REPARSE_LIMIT   1000
 
 
 #define        REPARSE_LIMIT   1000
 


@@ -81,7 +82,7 @@ static        const enum mandocerr    mandoclimits[MANDOCLEVEL_MAX] = {
        MANDOCERR_WARNING,
        MANDOCERR_WARNING,
        MANDOCERR_ERROR,
        MANDOCERR_WARNING,
        MANDOCERR_WARNING,
        MANDOCERR_ERROR,

-       MANDOCERR_FATAL,
+       MANDOCERR_UNSUPP,

        MANDOCERR_MAX,
        MANDOCERR_MAX
 };
        MANDOCERR_MAX,
        MANDOCERR_MAX
 };


@@ -97,7 +98,6 @@ static        const char * const      mandocerrs[MANDOCERR_MAX] = {
        "lower case character in document title",
        "missing manual section, using \"\"",
        "unknown manual section",
        "lower case character in document title",
        "missing manual section, using \"\"",
        "unknown manual section",

-       "unknown manual volume or arch",

        "missing date, using today's date",
        "cannot parse date, using it verbatim",
        "missing Os macro, using \"\"",
        "missing date, using today's date",
        "cannot parse date, using it verbatim",
        "missing Os macro, using \"\"",


@@ -111,7 +111,11 @@ static     const char * const      mandocerrs[MANDOCERR_MAX] = {
        "no document body",
        "content before first section header",
        "first section is not \"NAME\"",
        "no document body",
        "content before first section header",
        "first section is not \"NAME\"",

-       "bad NAME section contents",
+       "NAME section without name",
+       "NAME section without description",
+       "description not at the end of NAME",
+       "bad NAME section content",
+       "missing description line, using \"\"",

        "sections out of conventional order",
        "duplicate section title",
        "unexpected section",
        "sections out of conventional order",
        "duplicate section title",
        "unexpected section",


@@ -121,6 +125,7 @@ static      const char * const      mandocerrs[MANDOCERR_MAX] = {
 
        /* related to macros and nesting */
        "obsolete macro",
 
        /* related to macros and nesting */
        "obsolete macro",

+       "macro neither callable nor escaped",

        "skipping paragraph macro",
        "moving paragraph macro out of list",
        "skipping no-space macro",
        "skipping paragraph macro",
        "moving paragraph macro out of list",
        "skipping no-space macro",


@@ -136,17 +141,22 @@ static    const char * const      mandocerrs[MANDOCERR_MAX] = {
        "skipping empty request",
        "conditional request controls empty scope",
        "skipping empty macro",
        "skipping empty request",
        "conditional request controls empty scope",
        "skipping empty macro",

+       "empty block",

        "empty argument, using 0n",
        "empty argument, using 0n",

-       "argument count wrong",

        "missing display type, using -ragged",
        "list type is not the first argument",
        "missing -width in -tag list, using 8n",
        "missing utility name, using \"\"",
        "missing display type, using -ragged",
        "list type is not the first argument",
        "missing -width in -tag list, using 8n",
        "missing utility name, using \"\"",

+       "missing function name, using \"\"",

        "empty head in list item",
        "empty list item",
        "missing font type, using \\fR",
        "unknown font type, using \\fR",
        "empty head in list item",
        "empty list item",
        "missing font type, using \\fR",
        "unknown font type, using \\fR",

+       "nothing follows prefix",
+       "empty reference block",

        "missing -std argument, adding it",
        "missing -std argument, adding it",

+       "missing option string, using \"\"",
+       "missing resource identifier, using \"\"",

        "missing eqn box, using \"\"",
 
        /* related to bad macro arguments */
        "missing eqn box, using \"\"",
 
        /* related to bad macro arguments */


@@ -156,12 +166,14 @@ static    const char * const      mandocerrs[MANDOCERR_MAX] = {
        "skipping duplicate display type",
        "skipping duplicate list type",
        "skipping -width argument",
        "skipping duplicate display type",
        "skipping duplicate list type",
        "skipping -width argument",

+       "wrong number of cells",

        "unknown AT&T UNIX version",
        "comma in function argument",
        "parenthesis in function name",
        "invalid content in Rs block",
        "invalid Boolean argument",
        "unknown font, skipping request",
        "unknown AT&T UNIX version",
        "comma in function argument",
        "parenthesis in function name",
        "invalid content in Rs block",
        "invalid Boolean argument",
        "unknown font, skipping request",

+       "odd number of characters in request",

 
        /* related to plain text */
        "blank line in fill mode, using .sp",
 
        /* related to plain text */
        "blank line in fill mode, using .sp",


@@ -171,64 +183,60 @@ static    const char * const      mandocerrs[MANDOCERR_MAX] = {
        "invalid escape sequence",
        "undefined string, using \"\"",
 
        "invalid escape sequence",
        "undefined string, using \"\"",
 

-       "generic error",
+       /* related to tables */
+       "tbl line starts with span",
+       "tbl column starts with span",
+       "skipping vertical bar in tbl layout",

 
 

-       /* related to equations */
-       "unexpected equation scope closure",
-       "equation scope open on exit",
-       "overlapping equation scopes",
-       "unexpected end of equation",
+       "generic error",

 
        /* related to tables */
 
        /* related to tables */

-       "bad table syntax",
-       "bad table option",
-       "bad table layout",
-       "no table layout cells specified",
-       "no table data cells specified",
-       "ignore data in cell",
-       "data block still open",
-       "ignoring extra data cells",
+       "non-alphabetic character in tbl options",
+       "skipping unknown tbl option",
+       "missing tbl option argument",
+       "wrong tbl option argument size",
+       "empty tbl layout",
+       "invalid character in tbl layout",
+       "unmatched parenthesis in tbl layout",
+       "tbl without any data cells",
+       "ignoring data in spanned tbl cell",
+       "ignoring extra tbl data cells",
+       "data block open at end of tbl",

 
        /* related to document structure and macros */
 
        /* related to document structure and macros */

+       NULL,

        "input stack limit exceeded, infinite loop?",
        "skipping bad character",
        "skipping unknown macro",
        "input stack limit exceeded, infinite loop?",
        "skipping bad character",
        "skipping unknown macro",

+       "skipping insecure request",

        "skipping item outside list",
        "skipping column outside column list",
        "skipping end of block that is not open",
        "skipping item outside list",
        "skipping column outside column list",
        "skipping end of block that is not open",

+       "fewer RS blocks open, skipping",

        "inserting missing end of block",
        "appending missing end of block",
 
        /* related to request and macro arguments */
        "escaped character not allowed in a name",
        "inserting missing end of block",
        "appending missing end of block",
 
        /* related to request and macro arguments */
        "escaped character not allowed in a name",

-       "argument count wrong",
+       "NOT IMPLEMENTED: Bd -file",

        "missing list type, using -item",
        "missing manual name, using \"\"",
        "uname(3) system call failed, using UNKNOWN",
        "unknown standard specifier",
        "skipping request without numeric argument",
        "missing list type, using -item",
        "missing manual name, using \"\"",
        "uname(3) system call failed, using UNKNOWN",
        "unknown standard specifier",
        "skipping request without numeric argument",

+       "NOT IMPLEMENTED: .so with absolute path or \"..\"",
+       ".so request failed",

        "skipping all arguments",
        "skipping excess arguments",
        "divide by zero",
 
        "skipping all arguments",
        "skipping excess arguments",
        "divide by zero",
 

-       "generic fatal error",
-
+       "unsupported feature",

        "input too large",
        "input too large",

-       "NOT IMPLEMENTED: Bd -file",
-       "NOT IMPLEMENTED: .so with absolute path or \"..\"",
-       ".so request failed",
-
-       /* system errors */
-       "cannot dup file descriptor",
-       "cannot exec",
-       "gunzip failed with code",
-       "cannot fork",
-       NULL,
-       "cannot open pipe",
-       "cannot read file",
-       "gunzip died from signal",
-       "cannot stat file",
-       "wait failed",
+       "unsupported control character",
+       "unsupported roff request",
+       "eqn delim option in tbl",
+       "unsupported tbl layout modifier",
+       "ignoring macro in table",

 };
 
 static const char * const      mandoclevels[MANDOCLEVEL_MAX] = {
 };
 
 static const char * const      mandoclevels[MANDOCLEVEL_MAX] = {


@@ -236,7 +244,7 @@ static      const char * const      mandoclevels[MANDOCLEVEL_MAX] = {
        "RESERVED",
        "WARNING",
        "ERROR",
        "RESERVED",
        "WARNING",
        "ERROR",

-       "FATAL",
+       "UNSUPP",

        "BADARG",
        "SYSERR"
 };
        "BADARG",
        "SYSERR"
 };


@@ -297,7 +305,8 @@ choose_parser(struct mparse *curp)
        /* Fall back to man(7) as a last resort. */
 
        if (NULL == curp->pman)
        /* Fall back to man(7) as a last resort. */
 
        if (NULL == curp->pman)

-               curp->pman = man_alloc(curp->roff, curp,
+               curp->pman = man_alloc(
+                   curp->roff, curp, curp->defos,

                    MPARSE_QUICK & curp->options ? 1 : 0);
        assert(curp->pman);
        curp->man = curp->pman;
                    MPARSE_QUICK & curp->options ? 1 : 0);
        assert(curp->pman);
        curp->man = curp->pman;


@@ -315,10 +324,14 @@ mparse_buf_r(struct mparse *curp, struct buf blk, size_t i, int start)
 {
        const struct tbl_span   *span;
        struct buf       ln;
 {
        const struct tbl_span   *span;
        struct buf       ln;

+       const char      *save_file;
+       char            *cp;

        size_t           pos; /* byte number in the ln buffer */
        enum rofferr     rr;
        size_t           pos; /* byte number in the ln buffer */
        enum rofferr     rr;

-       int              of, rc;
+       int              of;

        int              lnn; /* line number in the real file */
        int              lnn; /* line number in the real file */

+       int              fd;
+       pid_t            save_child;

        unsigned char    c;
 
        memset(&ln, 0, sizeof(ln));
        unsigned char    c;
 
        memset(&ln, 0, sizeof(ln));


@@ -373,9 +386,8 @@ mparse_buf_r(struct mparse *curp, struct buf blk, size_t i, int start)
                        if (c & 0x80) {
                                if ( ! (curp->filenc && preconv_encode(
                                    &blk, &i, &ln, &pos, &curp->filenc))) {
                        if (c & 0x80) {
                                if ( ! (curp->filenc && preconv_encode(
                                    &blk, &i, &ln, &pos, &curp->filenc))) {

-                                       mandoc_vmsg(MANDOCERR_BADCHAR,
-                                           curp, curp->line, pos,
-                                           "0x%x", c);
+                                       mandoc_vmsg(MANDOCERR_CHAR_BAD, curp,
+                                           curp->line, pos, "0x%x", c);

                                        ln.buf[pos++] = '?';
                                        i++;
                                }
                                        ln.buf[pos++] = '?';
                                        i++;
                                }


@@ -387,10 +399,13 @@ mparse_buf_r(struct mparse *curp, struct buf blk, size_t i, int start)
                         */
 
                        if (c == 0x7f || (c < 0x20 && c != 0x09)) {
                         */
 
                        if (c == 0x7f || (c < 0x20 && c != 0x09)) {

-                               mandoc_vmsg(MANDOCERR_BADCHAR, curp,
-                                   curp->line, pos, "0x%x", c);
+                               mandoc_vmsg(c == 0x00 || c == 0x04 ||
+                                   c > 0x0a ? MANDOCERR_CHAR_BAD :
+                                   MANDOCERR_CHAR_UNSUPP,
+                                   curp, curp->line, pos, "0x%x", c);

                                i++;
                                i++;

-                               ln.buf[pos++] = '?';
+                               if (c != '\r')
+                                       ln.buf[pos++] = '?';

                                continue;
                        }
 
                                continue;
                        }
 


@@ -444,7 +459,7 @@ mparse_buf_r(struct mparse *curp, struct buf blk, size_t i, int start)
 
                        if ( ! (isascii(c) &&
                            (isgraph(c) || isblank(c)))) {
 
                        if ( ! (isascii(c) &&
                            (isgraph(c) || isblank(c)))) {

-                               mandoc_vmsg(MANDOCERR_BADCHAR, curp,
+                               mandoc_vmsg(MANDOCERR_CHAR_BAD, curp,

                                    curp->line, pos, "0x%x", c);
                                i += 2;
                                ln.buf[pos++] = '?';
                                    curp->line, pos, "0x%x", c);
                                i += 2;
                                ln.buf[pos++] = '?';


@@ -513,9 +528,6 @@ rerun:
                case ROFF_IGN:
                        pos = 0;
                        continue;
                case ROFF_IGN:
                        pos = 0;
                        continue;

-               case ROFF_ERR:
-                       assert(MANDOCLEVEL_FATAL <= curp->file_status);
-                       break;

                case ROFF_SO:
                        if ( ! (curp->options & MPARSE_SO) &&
                            (i >= blk.sz || blk.buf[i] == '\0')) {
                case ROFF_SO:
                        if ( ! (curp->options & MPARSE_SO) &&
                            (i >= blk.sz || blk.buf[i] == '\0')) {


@@ -530,27 +542,32 @@ rerun:
                         */
                        if (curp->secondary)
                                curp->secondary->sz -= pos + 1;
                         */
                        if (curp->secondary)
                                curp->secondary->sz -= pos + 1;

-                       mparse_readfd(curp, -1, ln.buf + of);
-                       if (MANDOCLEVEL_FATAL <= curp->file_status) {
+                       save_file = curp->file;
+                       save_child = curp->child;
+                       if (mparse_open(curp, &fd, ln.buf + of) ==
+                           MANDOCLEVEL_OK) {
+                               mparse_readfd(curp, fd, ln.buf + of);
+                               curp->file = save_file;
+                       } else {
+                               curp->file = save_file;

                                mandoc_vmsg(MANDOCERR_SO_FAIL,
                                    curp, curp->line, pos,
                                    ".so %s", ln.buf + of);
                                mandoc_vmsg(MANDOCERR_SO_FAIL,
                                    curp, curp->line, pos,
                                    ".so %s", ln.buf + of);

-                               break;
+                               ln.sz = mandoc_asprintf(&cp,
+                                   ".sp\nSee the file %s.\n.sp",
+                                   ln.buf + of);
+                               free(ln.buf);
+                               ln.buf = cp;
+                               of = 0;
+                               mparse_buf_r(curp, ln, of, 0);

                        }
                        }

+                       curp->child = save_child;

                        pos = 0;
                        continue;
                default:
                        break;
                }
 
                        pos = 0;
                        continue;
                default:
                        break;
                }
 

-               /*
-                * If we encounter errors in the recursive parse, make
-                * sure we don't continue parsing.
-                */
-
-               if (MANDOCLEVEL_FATAL <= curp->file_status)
-                       break;
-

                /*
                 * If input parsers have not been allocated, do so now.
                 * We keep these instanced between parsers, but set them
                /*
                 * If input parsers have not been allocated, do so now.
                 * We keep these instanced between parsers, but set them


@@ -570,34 +587,21 @@ rerun:
                 * Do the same for ROFF_EQN.
                 */
 
                 * Do the same for ROFF_EQN.
                 */
 

-               rc = -1;
-
-               if (ROFF_TBL == rr)
-                       while (NULL != (span = roff_span(curp->roff))) {
-                               rc = curp->man ?
-                                   man_addspan(curp->man, span) :
-                                   mdoc_addspan(curp->mdoc, span);
-                               if (0 == rc)
-                                       break;
-                       }
-               else if (ROFF_EQN == rr)
-                       rc = curp->mdoc ?
-                           mdoc_addeqn(curp->mdoc,
-                               roff_eqn(curp->roff)) :
-                           man_addeqn(curp->man,
-                               roff_eqn(curp->roff));
-               else if (curp->man || curp->mdoc)
-                       rc = curp->man ?
-                           man_parseln(curp->man,
-                               curp->line, ln.buf, of) :
-                           mdoc_parseln(curp->mdoc,
-                               curp->line, ln.buf, of);
-
-               if (0 == rc) {
-                       assert(MANDOCLEVEL_FATAL <= curp->file_status);
-                       break;
-               } else if (2 == rc)
-                       break;
+               if (rr == ROFF_TBL) {
+                       while ((span = roff_span(curp->roff)) != NULL)
+                               if (curp->man == NULL)
+                                       mdoc_addspan(curp->mdoc, span);
+                               else
+                                       man_addspan(curp->man, span);
+               } else if (rr == ROFF_EQN) {
+                       if (curp->man == NULL)
+                               mdoc_addeqn(curp->mdoc, roff_eqn(curp->roff));
+                       else
+                               man_addeqn(curp->man, roff_eqn(curp->roff));
+               } else if ((curp->man == NULL ?
+                   mdoc_parseln(curp->mdoc, curp->line, ln.buf, of) :
+                   man_parseln(curp->man, curp->line, ln.buf, of)) == 2)
+                               break;

 
                /* Temporary buffers typically are not full. */
 
 
                /* Temporary buffers typically are not full. */
 


@@ -622,11 +626,8 @@ read_whole_file(struct mparse *curp, const char *file, int fd,
 #if HAVE_MMAP
        struct stat      st;
        if (-1 == fstat(fd, &st)) {
 #if HAVE_MMAP
        struct stat      st;
        if (-1 == fstat(fd, &st)) {

-               curp->file_status = MANDOCLEVEL_SYSERR;
-               if (curp->mmsg)
-                       (*curp->mmsg)(MANDOCERR_SYSSTAT, curp->file_status,
-                           file, 0, 0, strerror(errno));
-               return(0);
+               perror(file);
+               exit((int)MANDOCLEVEL_SYSERR);

        }
 
        /*
        }
 
        /*


@@ -637,11 +638,8 @@ read_whole_file(struct mparse *curp, const char *file, int fd,
         */
 
        if (S_ISREG(st.st_mode)) {
         */
 
        if (S_ISREG(st.st_mode)) {

-               if (st.st_size >= (1U << 31)) {
-                       curp->file_status = MANDOCLEVEL_FATAL;
-                       if (curp->mmsg)
-                               (*curp->mmsg)(MANDOCERR_TOOLARGE,
-                                   curp->file_status, file, 0, 0, NULL);
+               if (st.st_size > 0x7fffffff) {
+                       mandoc_msg(MANDOCERR_TOOLARGE, curp, 0, 0, NULL);

                        return(0);
                }
                *with_mmap = 1;
                        return(0);
                }
                *with_mmap = 1;


@@ -664,11 +662,8 @@ read_whole_file(struct mparse *curp, const char *file, int fd,
        for (;;) {
                if (off == fb->sz) {
                        if (fb->sz == (1U << 31)) {
        for (;;) {
                if (off == fb->sz) {
                        if (fb->sz == (1U << 31)) {

-                               curp->file_status = MANDOCLEVEL_FATAL;
-                               if (curp->mmsg)
-                                       (*curp->mmsg)(MANDOCERR_TOOLARGE,
-                                           curp->file_status,
-                                           file, 0, 0, NULL);
+                               mandoc_msg(MANDOCERR_TOOLARGE, curp,
+                                   0, 0, NULL);

                                break;
                        }
                        resize_buf(fb, 65536);
                                break;
                        }
                        resize_buf(fb, 65536);


@@ -679,12 +674,8 @@ read_whole_file(struct mparse *curp, const char *file, int fd,
                        return(1);
                }
                if (ssz == -1) {
                        return(1);
                }
                if (ssz == -1) {

-                       curp->file_status = MANDOCLEVEL_SYSERR;
-                       if (curp->mmsg)
-                               (*curp->mmsg)(MANDOCERR_SYSREAD,
-                                   curp->file_status, file, 0, 0,
-                                   strerror(errno));
-                       break;
+                       perror(file);
+                       exit((int)MANDOCLEVEL_SYSERR);

                }
                off += (size_t)ssz;
        }
                }
                off += (size_t)ssz;
        }


@@ -698,9 +689,6 @@ static void
 mparse_end(struct mparse *curp)
 {
 
 mparse_end(struct mparse *curp)
 {
 

-       if (MANDOCLEVEL_FATAL <= curp->file_status)
-               return;
-

        if (curp->mdoc == NULL &&
            curp->man == NULL &&
            curp->sodest == NULL) {
        if (curp->mdoc == NULL &&
            curp->man == NULL &&
            curp->sodest == NULL) {


@@ -708,22 +696,16 @@ mparse_end(struct mparse *curp)
                        curp->mdoc = curp->pmdoc;
                else {
                        if (curp->pman == NULL)
                        curp->mdoc = curp->pmdoc;
                else {
                        if (curp->pman == NULL)

-                               curp->pman = man_alloc(curp->roff, curp,
+                               curp->pman = man_alloc(
+                                   curp->roff, curp, curp->defos,

                                    curp->options & MPARSE_QUICK ? 1 : 0);
                        curp->man = curp->pman;
                }
        }
                                    curp->options & MPARSE_QUICK ? 1 : 0);
                        curp->man = curp->pman;
                }
        }

-
-       if (curp->mdoc && ! mdoc_endparse(curp->mdoc)) {
-               assert(MANDOCLEVEL_FATAL <= curp->file_status);
-               return;
-       }
-
-       if (curp->man && ! man_endparse(curp->man)) {
-               assert(MANDOCLEVEL_FATAL <= curp->file_status);
-               return;
-       }
-
+       if (curp->mdoc)
+               mdoc_endparse(curp->mdoc);
+       if (curp->man)
+               man_endparse(curp->man);

        roff_endparse(curp->roff);
 }
 
        roff_endparse(curp->roff);
 }
 


@@ -760,7 +742,7 @@ mparse_parse_buffer(struct mparse *curp, struct buf blk, const char *file)
 
        mparse_buf_r(curp, blk, offset, 1);
 
 
        mparse_buf_r(curp, blk, offset, 1);
 

-       if (0 == --recursion_depth && MANDOCLEVEL_FATAL > curp->file_status)
+       if (--recursion_depth == 0)

                mparse_end(curp);
 
        curp->primary = svprimary;
                mparse_end(curp);
 
        curp->primary = svprimary;


@@ -768,18 +750,22 @@ mparse_parse_buffer(struct mparse *curp, struct buf blk, const char *file)
 }
 
 enum mandoclevel
 }
 
 enum mandoclevel

-mparse_readmem(struct mparse *curp, const void *buf, size_t len,
+mparse_readmem(struct mparse *curp, void *buf, size_t len,

                const char *file)
 {
        struct buf blk;
 
                const char *file)
 {
        struct buf blk;
 

-       blk.buf = UNCONST(buf);
+       blk.buf = buf;

        blk.sz = len;
 
        mparse_parse_buffer(curp, blk, file);
        return(curp->file_status);
 }
 
        blk.sz = len;
 
        mparse_parse_buffer(curp, blk, file);
        return(curp->file_status);
 }
 

+/*
+ * Read the whole file into memory and call the parsers.
+ * Called recursively when an .so request is encountered.
+ */

 enum mandoclevel
 mparse_readfd(struct mparse *curp, int fd, const char *file)
 {
 enum mandoclevel
 mparse_readfd(struct mparse *curp, int fd, const char *file)
 {


@@ -787,22 +773,6 @@ mparse_readfd(struct mparse *curp, int fd, const char *file)
        int              with_mmap;
        int              save_filenc;
 
        int              with_mmap;
        int              save_filenc;
 

-       if (-1 == fd && -1 == (fd = open(file, O_RDONLY, 0))) {
-               curp->file_status = MANDOCLEVEL_SYSERR;
-               if (curp->mmsg)
-                       (*curp->mmsg)(MANDOCERR_SYSOPEN,
-                           curp->file_status,
-                           file, 0, 0, strerror(errno));
-               return(curp->file_status);
-       }
-
-       /*
-        * Run for each opened file; may be called more than once for
-        * each full parse sequence if the opened file is nested (i.e.,
-        * from `so').  Simply sucks in the whole file and moves into
-        * the parse phase for the file.
-        */
-

        if (read_whole_file(curp, file, fd, &blk, &with_mmap)) {
                save_filenc = curp->filenc;
                curp->filenc = curp->options &
        if (read_whole_file(curp, file, fd, &blk, &with_mmap)) {
                save_filenc = curp->filenc;
                curp->filenc = curp->options &


@@ -817,9 +787,10 @@ mparse_readfd(struct mparse *curp, int fd, const char *file)
                        free(blk.buf);
        }
 
                        free(blk.buf);
        }
 

-       if (STDIN_FILENO != fd && -1 == close(fd))
+       if (fd != STDIN_FILENO && close(fd) == -1)

                perror(file);
 
                perror(file);
 

+       mparse_wait(curp);

        return(curp->file_status);
 }
 
        return(curp->file_status);
 }
 


@@ -827,58 +798,65 @@ enum mandoclevel
 mparse_open(struct mparse *curp, int *fd, const char *file)
 {
        int               pfd[2];
 mparse_open(struct mparse *curp, int *fd, const char *file)
 {
        int               pfd[2];

+       int               save_errno;

        char             *cp;
        char             *cp;

-       enum mandocerr    err;

 
 

-       pfd[1] = -1;

        curp->file = file;
        curp->file = file;

+
+       /* Unless zipped, try to just open the file. */
+

        if ((cp = strrchr(file, '.')) == NULL ||
            strcmp(cp + 1, "gz")) {
                curp->child = 0;
        if ((cp = strrchr(file, '.')) == NULL ||
            strcmp(cp + 1, "gz")) {
                curp->child = 0;

-               if ((*fd = open(file, O_RDONLY)) == -1) {
-                       err = MANDOCERR_SYSOPEN;
-                       goto out;
-               }
-               return(MANDOCLEVEL_OK);
+               if ((*fd = open(file, O_RDONLY)) != -1)
+                       return(MANDOCLEVEL_OK);
+
+               /* Open failed; try to append ".gz". */
+
+               mandoc_asprintf(&cp, "%s.gz", file);
+               file = cp;
+       } else
+               cp = NULL;
+
+       /* Before forking, make sure the file can be read. */
+
+       save_errno = errno;
+       if (access(file, R_OK) == -1) {
+               if (cp != NULL)
+                       errno = save_errno;
+               free(cp);
+               *fd = -1;
+               curp->child = 0;
+               mandoc_msg(MANDOCERR_FILE, curp, 0, 0, strerror(errno));
+               return(MANDOCLEVEL_ERROR);

        }
 
        }
 

+       /* Run gunzip(1). */
+

        if (pipe(pfd) == -1) {
        if (pipe(pfd) == -1) {

-               err = MANDOCERR_SYSPIPE;
-               goto out;
+               perror("pipe");
+               exit((int)MANDOCLEVEL_SYSERR);

        }
 
        switch (curp->child = fork()) {
        case -1:
        }
 
        switch (curp->child = fork()) {
        case -1:

-               err = MANDOCERR_SYSFORK;
-               close(pfd[0]);
-               close(pfd[1]);
-               pfd[1] = -1;
-               break;
+               perror("fork");
+               exit((int)MANDOCLEVEL_SYSERR);

        case 0:
                close(pfd[0]);
                if (dup2(pfd[1], STDOUT_FILENO) == -1) {
        case 0:
                close(pfd[0]);
                if (dup2(pfd[1], STDOUT_FILENO) == -1) {

-                       err = MANDOCERR_SYSDUP;
-                       break;
+                       perror("dup");
+                       exit((int)MANDOCLEVEL_SYSERR);

                }
                }

+               signal(SIGPIPE, SIG_DFL);

                execlp("gunzip", "gunzip", "-c", file, NULL);
                execlp("gunzip", "gunzip", "-c", file, NULL);

-               err = MANDOCERR_SYSEXEC;
-               break;
+               perror("exec");
+               exit((int)MANDOCLEVEL_SYSERR);

        default:
                close(pfd[1]);
                *fd = pfd[0];
                return(MANDOCLEVEL_OK);
        }
        default:
                close(pfd[1]);
                *fd = pfd[0];
                return(MANDOCLEVEL_OK);
        }

-
-out:
-       *fd = -1;
-       curp->child = 0;
-       curp->file_status = MANDOCLEVEL_SYSERR;
-       if (curp->mmsg)
-               (*curp->mmsg)(err, curp->file_status, file,
-                   0, 0, strerror(errno));
-       if (pfd[1] != -1)
-               exit(1);
-       return(curp->file_status);

 }
 
 enum mandoclevel
 }
 
 enum mandoclevel


@@ -890,22 +868,19 @@ mparse_wait(struct mparse *curp)
                return(MANDOCLEVEL_OK);
 
        if (waitpid(curp->child, &status, 0) == -1) {
                return(MANDOCLEVEL_OK);
 
        if (waitpid(curp->child, &status, 0) == -1) {

-               mandoc_msg(MANDOCERR_SYSWAIT, curp, 0, 0,
-                   strerror(errno));
-               curp->file_status = MANDOCLEVEL_SYSERR;
-               return(curp->file_status);
+               perror("wait");
+               exit((int)MANDOCLEVEL_SYSERR);

        }
        }

+       curp->child = 0;

        if (WIFSIGNALED(status)) {
        if (WIFSIGNALED(status)) {

-               mandoc_vmsg(MANDOCERR_SYSSIG, curp, 0, 0,
-                   "%d", WTERMSIG(status));
-               curp->file_status = MANDOCLEVEL_SYSERR;
-               return(curp->file_status);
+               mandoc_vmsg(MANDOCERR_FILE, curp, 0, 0,
+                   "gunzip died from signal %d", WTERMSIG(status));
+               return(MANDOCLEVEL_ERROR);

        }
        if (WEXITSTATUS(status)) {
        }
        if (WEXITSTATUS(status)) {

-               mandoc_vmsg(MANDOCERR_SYSEXIT, curp, 0, 0,
-                   "%d", WEXITSTATUS(status));
-               curp->file_status = MANDOCLEVEL_SYSERR;
-               return(curp->file_status);
+               mandoc_vmsg(MANDOCERR_FILE, curp, 0, 0,
+                   "gunzip failed with code %d", WEXITSTATUS(status));
+               return(MANDOCLEVEL_ERROR);

        }
        return(MANDOCLEVEL_OK);
 }
        }
        return(MANDOCLEVEL_OK);
 }


@@ -916,8 +891,6 @@ mparse_alloc(int options, enum mandoclevel wlevel, mandocmsg mmsg,
 {
        struct mparse   *curp;
 
 {
        struct mparse   *curp;
 

-       assert(wlevel <= MANDOCLEVEL_FATAL);
-

        curp = mandoc_calloc(1, sizeof(struct mparse));
 
        curp->options = options;
        curp = mandoc_calloc(1, sizeof(struct mparse));
 
        curp->options = options;


@@ -932,7 +905,8 @@ mparse_alloc(int options, enum mandoclevel wlevel, mandocmsg mmsg,
                    curp->roff, curp, curp->defos,
                    curp->options & MPARSE_QUICK ? 1 : 0);
        if (curp->options & MPARSE_MAN)
                    curp->roff, curp, curp->defos,
                    curp->options & MPARSE_QUICK ? 1 : 0);
        if (curp->options & MPARSE_MAN)

-               curp->pman = man_alloc(curp->roff, curp,
+               curp->pman = man_alloc(
+                   curp->roff, curp, curp->defos,

                    curp->options & MPARSE_QUICK ? 1 : 0);
 
        return(curp);
                    curp->options & MPARSE_QUICK ? 1 : 0);
 
        return(curp);


@@ -1013,11 +987,11 @@ mandoc_msg(enum mandocerr er, struct mparse *m,
 {
        enum mandoclevel level;
 
 {
        enum mandoclevel level;
 

-       level = MANDOCLEVEL_FATAL;
+       level = MANDOCLEVEL_UNSUPP;

        while (er < mandoclimits[level])
                level--;
 
        while (er < mandoclimits[level])
                level--;
 

-       if (level < m->wlevel)
+       if (level < m->wlevel && er != MANDOCERR_FILE)

                return;
 
        if (m->mmsg)
                return;
 
        if (m->mmsg)