diff options
| author |   Ingo Schwarze <schwarze@openbsd.org> | 2012-07-12 15:11:14 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@openbsd.org> | 2012-07-12 15:11:14 +0000 |
| commit | a19c95fa6a0579b54387165ee1396f52a4f586df (patch) | |
| tree | fe4387c9b2dd40fe521806867f53388a88512b1b /read.c | |
| parent | 5b96439af7b14f4fc13c1aa3249974a9ad9ddfd4 (diff) | |
| download | mandoc-a19c95fa6a0579b54387165ee1396f52a4f586df.tar.gz mandoc-a19c95fa6a0579b54387165ee1396f52a4f586df.tar.zst mandoc-a19c95fa6a0579b54387165ee1396f52a4f586df.zip | |
The post_nm() validation function crashed when the first .Nm child node
was a non-text node. Fix this by rewriting post_nm() to always set
the meta name to UNKNOWN when the name is missing or unusable.
While here, make MANDOCERR_NONAME an ERROR, as it usually renders
the page content unintelligible.
Bug reported by Maxim <Belooussov at gmail dot com>, thanks.
OpenBSD rev. 1.105
Diffstat (limited to 'read.c')
| -rw-r--r-- | read.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -1,4 +1,4 @@ -/* $Id: read.c,v 1.29 2012/05/27 17:48:57 schwarze Exp $ */ +/* $Id: read.c,v 1.30 2012/07/12 15:11:14 schwarze Exp $ */ /* * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010, 2011 Ingo Schwarze <schwarze@openbsd.org> @@ -106,7 +106,6 @@ static const char * const mandocerrs[MANDOCERR_MAX] = { ".so is fragile, better use ln(1)", "NAME section must come first", "bad NAME section contents", - "manual name not yet set", "sections out of conventional order", "duplicate section name", "section not in conventional manual section", @@ -174,6 +173,7 @@ static const char * const mandocerrs[MANDOCERR_MAX] = { "input stack limit exceeded, infinite loop?", "skipping bad character", "escaped character not allowed in a name", + "manual name not yet set", "skipping text before the first section header", "skipping unknown macro", "NOT IMPLEMENTED, please use groff: skipping request", |