diff options
| author | Ingo Schwarze <schwarze@openbsd.org> | 2011-12-04 00:44:12 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@openbsd.org> | 2011-12-04 00:44:12 +0000 |
| commit | 104a65153587e466515ddab8bd99fccb419f651c (patch) | |
| tree | a22de01e1885c82fe14a74a4ed7ce747b2d4c6b9 /man_macro.c | |
| parent | 1e7720d904665f833840ea9ab53b3879b399fcc4 (diff) | |
| download | mandoc-104a65153587e466515ddab8bd99fccb419f651c.tar.gz | |
Jumping out of man_unscope() for the root node is a bad idea
because that will skip root node validation, potentially entering
rendering modules will NULL pointers lurking in the meta data.
Instead, always validate the root node and (as suggested by joerg@)
assert validity of the meta data before using it in the renderers.
ok joerg@
Diffstat (limited to 'man_macro.c')
| -rw-r--r-- | man_macro.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/man_macro.c b/man_macro.c index 42d42279..50dd477d 100644 --- a/man_macro.c +++ b/man_macro.c @@ -120,8 +120,6 @@ man_unscope(struct man *m, const struct man_node *to, assert(to); - if (MAN_ROOT == m->last->type) - return(1); m->next = MAN_NEXT_SIBLING; /* LINTED */ |