fnematch: fix out-of-bounds access on EOF
authormillert <millert@openbsd.org>
Wed, 15 Nov 2023 18:48:13 +0000 (18:48 +0000)
committermillert <millert@openbsd.org>
Wed, 15 Nov 2023 18:48:13 +0000 (18:48 +0000)
commit8f894c94b7d9a5f934f92b31a8e4ca06fb0b8863
tree11f57062a2d4d37d8216ca5c25756431c6fd47bd
parentc338e3e2ad38b099fa13a573170a235f9ed893ce
fnematch: fix out-of-bounds access on EOF

fnematch() expects to store a NUL byte when EOF is encountered.
However, the rewrite broke this assumption because r.len from getrune()
is zero on EOF.  This results in j becoming negative on EOF, causing an
out-of-bounds access.  It is simplest to just force r.len to 1 on EOF
to copy a single NUL byte--the rune is initialized to zero even for EOF.

This also fixes the call to adjbuf().  We cannot use 'k' to determine
when we need to expand the buffer now that we are potentially reading
more than a single byte at a time.

https://github.com/onetrueawk/awk/pull/211
usr.bin/awk/b.c