diff options
| author |   schwarze <schwarze@openbsd.org> | 2016-08-11 10:46:27 +0000 |
|---|---|---|
| committer | schwarze <schwarze@openbsd.org> | 2016-08-11 10:46:27 +0000 |
| commit | 4b5c4138fd2b4586b5e417244fa9b92c108ecc30 (patch) | |
| tree | 4dba30066bfbb6a32e2f0d69f8d5fdddca0c8c1f /usr.bin/mandoc/mdoc_validate.c | |
| parent | fuse requires ufs_ihash.c (diff) | |
| download | wireguard-openbsd-4b5c4138fd2b4586b5e417244fa9b92c108ecc30.tar.xz wireguard-openbsd-4b5c4138fd2b4586b5e417244fa9b92c108ecc30.zip | |
If a .Bd display is on the one hand doomed to be deleted because
it has no type, but is on the other hand breaking another block,
delete its end marker as well, or the end marker may remain behind
as an orphan, triggering an assertion in the terminal formatter.
Problem found by tb@ with afl(1).
Diffstat (limited to 'usr.bin/mandoc/mdoc_validate.c')
| -rw-r--r-- | usr.bin/mandoc/mdoc_validate.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/usr.bin/mandoc/mdoc_validate.c b/usr.bin/mandoc/mdoc_validate.c index 3100a79794c..77524485658 100644 --- a/usr.bin/mandoc/mdoc_validate.c +++ b/usr.bin/mandoc/mdoc_validate.c @@ -1,4 +1,4 @@ -/* $OpenBSD: mdoc_validate.c,v 1.221 2016/08/10 20:16:43 schwarze Exp $ */ +/* $OpenBSD: mdoc_validate.c,v 1.222 2016/08/11 10:46:27 schwarze Exp $ */ /* * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010-2016 Ingo Schwarze <schwarze@openbsd.org> @@ -877,9 +877,10 @@ post_display(POST_ARGS) n = mdoc->last; switch (n->type) { case ROFFT_BODY: - if (n->end != ENDBODY_NOT) - break; - if (n->child == NULL) + if (n->end != ENDBODY_NOT) { + if (n->tok == MDOC_Bd && n->parent->args == NULL) + roff_node_delete(mdoc, n); + } else if (n->child == NULL) mandoc_msg(MANDOCERR_BLK_EMPTY, mdoc->parse, n->line, n->pos, mdoc_macronames[n->tok]); else if (n->tok == MDOC_D1) |