]> git.ipfire.org Git - thirdparty/git.git/commit - git-compat-util.h
pretty: fix integer overflow in wrapping format
authorPatrick Steinhardt <ps@pks.im>
Thu, 1 Dec 2022 14:46:49 +0000 (15:46 +0100)
committerJunio C Hamano <gitster@pobox.com>
Fri, 9 Dec 2022 05:26:21 +0000 (14:26 +0900)
commit48050c42c73c28b0c001d63d11dffac7e116847b
treefbea2ce85a3b749345fd987b13277e8f7ab26e00
parent1de69c0cdd388b0a5b7bdde0bfa0bda514a354b0
pretty: fix integer overflow in wrapping format

The `%w(width,indent1,indent2)` formatting directive can be used to
rewrap text to a specific width and is designed after git-shortlog(1)'s
`-w` parameter. While the three parameters are all stored as `size_t`
internally, `strbuf_add_wrapped_text()` accepts integers as input. As a
result, the casted integers may overflow. As these now-negative integers
are later on passed to `strbuf_addchars()`, we will ultimately run into
implementation-defined behaviour due to casting a negative number back
to `size_t` again. On my platform, this results in trying to allocate
9000 petabyte of memory.

Fix this overflow by using `cast_size_t_to_int()` so that we reject
inputs that cannot be represented as an integer.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-compat-util.h
pretty.c
t/t4205-log-pretty-formats.sh