From 882b0df046673132c4d2d6f14969d087ead1d87c Mon Sep 17 00:00:00 2001 From: Ingo Schwarze Date: Wed, 13 Apr 2022 14:45:50 +0000 Subject: Do not die on an assertion if an input file contains no section whatsoever and ends with a broken next-line scope. Obviously, this cannot happen in a real manual page, but mandoc(1) should not die even when fed absurd input. This bug was independently reported by both jsg@ and tb@ who both found it with afl(1). --- man_macro.c | 5 +++-- regress/man/TH/nobody.in | 5 +++-- regress/man/TH/nobody.out_ascii | 2 +- regress/man/TH/nobody.out_lint | 1 + 4 files changed, 8 insertions(+), 5 deletions(-) diff --git a/man_macro.c b/man_macro.c index b040823a..e16a6a74 100644 --- a/man_macro.c +++ b/man_macro.c @@ -1,7 +1,7 @@ /* $Id$ */ /* + * Copyright (c) 2012-2015,2017-2020,2022 Ingo Schwarze * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons - * Copyright (c) 2012-2015, 2017-2020 Ingo Schwarze * Copyright (c) 2013 Franco Fichtner * * Permission to use, copy, modify, and distribute this software for any @@ -108,7 +108,8 @@ man_unscope(struct roff_man *man, const struct roff_node *to) n->line, n->pos, "EOF breaks %s", roff_name[n->tok]); if (man->flags & MAN_ELINE) { - if ((man_macro(n->parent->tok)->flags & + if (n->parent->type == ROFFT_ROOT || + (man_macro(n->parent->tok)->flags & MAN_ESCOPED) == 0) man->flags &= ~MAN_ELINE; } else { diff --git a/regress/man/TH/nobody.in b/regress/man/TH/nobody.in index b24909ad..5ef4c1d4 100644 --- a/regress/man/TH/nobody.in +++ b/regress/man/TH/nobody.in @@ -1,2 +1,3 @@ -.\" $OpenBSD: nobody.in,v 1.2 2017/07/04 14:53:24 schwarze Exp $ -.TH TH-NOBODY 1 "June 20, 2014" +.\" $OpenBSD: nobody.in,v 1.3 2022/04/13 14:37:34 schwarze Exp $ +.TH TH-NOBODY 1 "April 13, 2022" +.B diff --git a/regress/man/TH/nobody.out_ascii b/regress/man/TH/nobody.out_ascii index 2b0b1ed2..975e8a77 100644 --- a/regress/man/TH/nobody.out_ascii +++ b/regress/man/TH/nobody.out_ascii @@ -1,3 +1,3 @@ TH-NOBODY(1) General Commands Manual TH-NOBODY(1) -OpenBSD June 20, 2014 TH-NOBODY(1) +OpenBSD April 13, 2022 TH-NOBODY(1) diff --git a/regress/man/TH/nobody.out_lint b/regress/man/TH/nobody.out_lint index f5439fcd..dae5fdf0 100644 --- a/regress/man/TH/nobody.out_lint +++ b/regress/man/TH/nobody.out_lint @@ -1 +1,2 @@ +mandoc: nobody.in:3:2: WARNING: line scope broken: EOF breaks B mandoc: nobody.in: WARNING: no document body -- cgit