fix FILE buffer underflow in ungetwc
authorRich Felker <dalias@aerifal.cx>
Tue, 26 Apr 2016 19:26:40 +0000 (15:26 -0400)
committerRich Felker <dalias@aerifal.cx>
Tue, 26 Apr 2016 19:26:40 +0000 (15:26 -0400)
commit6ed791e768d83b40ed56c99dbb1ed72c1e49aae7
treea31203a6ee0fe3131e4366867e5c7f48cbf58b45
parentbe999f7a54e81656ea0409030205177327a7450c
fix FILE buffer underflow in ungetwc

commit 7e816a6487932cbb3cb71d94b609e50e81f4e5bf (version 1.1.11
release cycle) moved the code that performs wchar_t to multibyte
conversion across code that used the resulting length in bytes,
thereby breaking the unget buffer space check in ungetwc and
clobbering up to three bytes below the start of the buffer.

for allocated FILEs (all read-enabled FILEs except stdin), the
underflow clobbers at most the FILE-specific locale pointer. no stores
are performed through this pointer, but subsequent loads may result in
a crash or mismatching encoding rule (UTF-8 multibyte vs byte-based).

for stdin, the buffer lies in .bss and the underflow may clobber
another object. in practice, for libc.so the adjacent object seems to
be stderr's buffer, which is completely unused, but this could vary
with linking options, or when static linking.

applications which do not attempt to use more than one character of
ungetwc pushback, or which do not use ungetwc, are not affected.
src/stdio/ungetwc.c