summaryrefslogtreecommitdiff
path: root/distrib
diff options
context:
space:
mode:
authorIngo Schwarze <schwarze@cvs.openbsd.org>2018-12-21 16:58:50 +0000
committerIngo Schwarze <schwarze@cvs.openbsd.org>2018-12-21 16:58:50 +0000
commit7928109223463f24bfe9976dc7a663ec40d4f32f (patch)
tree6973d5b6b42df71093cd3b7fb8c0cc65af9c1045 /distrib
parent69025f72dda5ad51f12ce2e464aa69ff47385c5f (diff)
Rename mandoc_getarg() to roff_getarg() and pass it the roff parser
struct as an argument such that after copy-in, it can call roff_expand() once again, which used to be called roff_res() before this. This fixes a subtle low-level roff(7) parsing bug reported by Fabio Scotoni <fabio at esse dot ch> in the 4.4BSD-Lite2 mdoc.samples(7) manual page, because that page used an escaped escape sequence in a macro argument. To expand escaped escape sequences in quoted mdoc(7) arguments, too, stop bypassing the call to roff_getarg() in mdoc_argv.c, function args() for this case. This does not solve the case of escaped escape sequences in quoted .Bl -column phrases yet. Because roff_expand() can make the string longer, roff_getarg() can no longer operate in-place but needs to malloc(3) the returned string. In the high-level parsers, free(3) that string after processing it.
Diffstat (limited to 'distrib')
0 files changed, 0 insertions, 0 deletions