aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/mdoc_validate.c
diff options
context:
space:
mode:
authorIngo Schwarze <schwarze@openbsd.org>2019-09-13 19:26:46 +0000
committerIngo Schwarze <schwarze@openbsd.org>2019-09-13 19:26:46 +0000
commit333417082dab30264386c33095d1f6e64f6939cc (patch)
tree848b1ddd19c1d1104764ca4b8dfd818c6e77d90a /mdoc_validate.c
parentf6bc3c824b41837a8b513e2805d3fd8a638f9912 (diff)
downloadmandoc-333417082dab30264386c33095d1f6e64f6939cc.tar.gz
mandoc-333417082dab30264386c33095d1f6e64f6939cc.tar.zst
mandoc-333417082dab30264386c33095d1f6e64f6939cc.zip
Improve validation of function names:
1. Relax checking to accept function types of the form "ret_type (fname)(args)" (suggested by Yuri Pankov <yuripv dot net>). 2. Tighten checking to require the closing parenthesis.
Diffstat (limited to 'mdoc_validate.c')
-rw-r--r--mdoc_validate.c18
1 files changed, 12 insertions, 6 deletions
diff --git a/mdoc_validate.c b/mdoc_validate.c
index 11cdf00b..c14d4008 100644
--- a/mdoc_validate.c
+++ b/mdoc_validate.c
@@ -1,4 +1,4 @@
-/* $Id: mdoc_validate.c,v 1.374 2019/06/27 15:07:30 schwarze Exp $ */
+/* $Id: mdoc_validate.c,v 1.375 2019/09/13 19:26:46 schwarze Exp $ */
/*
* Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv>
* Copyright (c) 2010-2019 Ingo Schwarze <schwarze@openbsd.org>
@@ -1186,11 +1186,17 @@ post_fname(POST_ARGS)
size_t pos;
n = mdoc->last->child;
- pos = strcspn(n->string, "()");
- cp = n->string + pos;
- if ( ! (cp[0] == '\0' || (cp[0] == '(' && cp[1] == '*')))
- mandoc_msg(MANDOCERR_FN_PAREN, n->line, n->pos + pos,
- "%s", n->string);
+ cp = n->string;
+ if (*cp == '(') {
+ if (cp[strlen(cp + 1)] == ')')
+ return;
+ pos = 0;
+ } else {
+ pos = strcspn(cp, "()");
+ if (cp[pos] == '\0')
+ return;
+ }
+ mandoc_msg(MANDOCERR_FN_PAREN, n->line, n->pos + pos, "%s", cp);
}
static void