]> git.cameronkatri.com Git - mandoc.git/commit
To prevent infinite recursion while expanding eqn(7) definitions,
authorIngo Schwarze <schwarze@openbsd.org>
Wed, 13 Apr 2022 20:26:19 +0000 (20:26 +0000)
committerIngo Schwarze <schwarze@openbsd.org>
Wed, 13 Apr 2022 20:26:19 +0000 (20:26 +0000)
commit10729b3aa1181d3f40ecb3ba0f955f96d86eaee3
tree23e0f420588718bcd80be303d809a557ee100167
parent4062479d6729bdfb0de4954e7a413ca3282d17cd
To prevent infinite recursion while expanding eqn(7) definitions,
we must not reset the recursion counter when moving beyond the end
of the *previous* expansion, but we may only do so when moving
beyond the rightmost position reached by *any* expansion in the
current equation.  This matters because definitions can nest;
consider:

.EQ
define inner "content"
define outer "inner outer"
outer
.EN

This endless loop was found by tb@ using afl(1).

Incidentally, GNU eqn(1) also performs an infinite loop in this
situation and then crashes when memory runs out, but that's not an
excuse for nasty behaviour of mandoc(1).

While here, consistently print the expanded content even when the
expansion is finally truncated.  While that is not likely to help
end-users, it may help authors of eqn(7) code to understand what's
going on.  Besides, it sends a very clear signal that something is
amiss, which was easy to miss in the past unless people
enabled -W error or used -T lint.
eqn.c
eqn_parse.h
regress/eqn/define/infinite.in
regress/eqn/define/infinite.out_ascii
regress/eqn/define/infinite.out_lint