From a350521d8151422ac228e507357343fe53d34ea1 Mon Sep 17 00:00:00 2001 From: Ingo Schwarze Date: Tue, 25 Nov 2014 20:00:01 +0000 Subject: Do not access a NULL pointer when a section has no body, which can for example happen for .Sh Xo .Sh without .Xc. Crash found by jsg@ with afl. --- mdoc_macro.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'mdoc_macro.c') diff --git a/mdoc_macro.c b/mdoc_macro.c index acc9c68c..e62b76cb 100644 --- a/mdoc_macro.c +++ b/mdoc_macro.c @@ -437,9 +437,11 @@ rew_dohalt(enum mdoct tok, enum mdoc_type type, * Default block rewinding rules. * In particular, always skip block end markers, * and let all blocks rewind Nm children. + * Do not warn again when closing a block, + * since closing the body already warned. */ if (ENDBODY_NOT != p->end || MDOC_Nm == p->tok || - (MDOC_BLOCK == p->type && + MDOC_BLOCK == type || (MDOC_BLOCK == p->type && ! (MDOC_EXPLICIT & mdoc_macros[tok].flags))) return(REWIND_MORE); -- cgit