diff options
| author | Ingo Schwarze <schwarze@openbsd.org> | 2022-06-06 12:09:48 +0000 |
|---|---|---|
| committer | Ingo Schwarze <schwarze@openbsd.org> | 2022-06-06 12:09:48 +0000 |
| commit | 1dd6aedb4115e5f45495a845aa50a5b0695ba9aa (patch) | |
| tree | 4860b1d46181e6922b7923884000d35ad1068f20 | |
| parent | ccc0a13d6e8c4072fb6381ba63162395bab8c78b (diff) | |
| download | mandoc-1dd6aedb4115e5f45495a845aa50a5b0695ba9aa.tar.gz | |
Allow arbitrary argument delimiters for \C, like groff does.
The restriction of only allowing ' as the delimiter was introduced
by kristaps@ on 2011/04/09 when he first supported \C.
For most other escape sequences, similar restrictions were relaxed
later on, but for the rarely used \C, it was apparently forgotten.
While here, reject empty character names: they are never valid.
| -rw-r--r-- | roff_escape.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/roff_escape.c b/roff_escape.c index 777d753b..39d57e88 100644 --- a/roff_escape.c +++ b/roff_escape.c @@ -206,10 +206,6 @@ roff_escape(const char *buf, const int ln, const int aesc, term = '\b'; break; case 'C': - if (buf[iarg] != '\'') { - rval = ESCAPE_ERROR; - goto out; - } rval = ESCAPE_SPECIAL; term = '\b'; break; @@ -391,6 +387,11 @@ roff_escape(const char *buf, const int ln, const int aesc, break; case ESCAPE_SPECIAL: + if (argl == 0) { + err = MANDOCERR_ESC_BADCHAR; + rval = ESCAPE_ERROR; + break; + } /* * The file chars.c only provides one common list of |