diff options
| author |   Ingo Schwarze <schwarze@openbsd.org> | 2015-03-15 16:53:41 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@openbsd.org> | 2015-03-15 16:53:41 +0000 |
| commit | c7e9eaf82d8a0689a69d341a31f3ba3206e20bc3 (patch) | |
| tree | 83f55242658757863eae69bdd7d3286fa85631b7 /man_term.c | |
| parent | 57952d3568fc684859e0e7a04e942d11736c1792 (diff) | |
| download | mandoc-c7e9eaf82d8a0689a69d341a31f3ba3206e20bc3.tar.gz mandoc-c7e9eaf82d8a0689a69d341a31f3ba3206e20bc3.tar.zst mandoc-c7e9eaf82d8a0689a69d341a31f3ba3206e20bc3.zip | |
Avoid off-by-one read access to the termacts array, which could
sometimes result in missing line breaks before subsection headers.
Found by carsten dot kunze at arcor dot de on SuSE 13.2.
Diffstat (limited to 'man_term.c')
| -rw-r--r-- | man_term.c | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -1,4 +1,4 @@ -/* $Id: man_term.c,v 1.169 2015/03/06 15:48:52 schwarze Exp $ */ +/* $Id: man_term.c,v 1.170 2015/03/15 16:53:41 schwarze Exp $ */ /* * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010-2015 Ingo Schwarze <schwarze@openbsd.org> @@ -743,7 +743,8 @@ pre_SS(DECL_ARGS) do { n = n->prev; - } while (n != NULL && termacts[n->tok].flags & MAN_NOTEXT); + } while (n != NULL && n->tok != MAN_MAX && + termacts[n->tok].flags & MAN_NOTEXT); if (n == NULL || (n->tok == MAN_SS && n->body->child == NULL)) break; |