]> granicus.if.org Git - icu/commit
ICU-20362 segfault/leftover files with long lines
authorTobias Stoeckmann <tobias@stoeckmann.org>
Sat, 19 Jan 2019 13:24:56 +0000 (14:24 +0100)
committerSteven R. Loomis <srl295@gmail.com>
Wed, 13 Feb 2019 18:23:19 +0000 (10:23 -0800)
commit7369eff4eddf3a83c03f25eaaf73ee33f46b4499
tree1772a7f70ad44eba51266816a9c36ae61a954bcd
parentc04f9f1c01d1f36d8c47fdc0e233d253014d236d
ICU-20362 segfault/leftover files with long lines

If a file with an input line larger than INT32_MAX (i.e. 2 GB) contains
an UTF8 character after that limit, escapesrc crashes on 64 bit systems
or does not remove incomplete files on 32 bit systems.

The issue is that an unchecked cast from size_t to int32_t can turn
negative, which results in negative offsets during array access.

This will eventually lead to an out of boundary read, which most likely
crashes the tool.

This patch sets a fixed limit on 1 GB to make sure that no side effects
occur if the line is exactly INT32_MAX or a few bytes less. It should
still be way more than anyone would really need.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
icu4c/source/tools/escapesrc/escapesrc.cpp