diff options
author | Joey Hess <id@joeyh.name> | 2019-03-07 14:56:57 -0500 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2019-03-08 10:13:00 +0900 |
commit | 02156ab031e430bc45ce6984dfc712de9962dec8 (patch) | |
tree | db16444a4187743f57576cd91a5be03ff58bd5f5 /builtin/verify-tag.c | |
parent | Git 2.19.2 (diff) | |
download | tgif-02156ab031e430bc45ce6984dfc712de9962dec8.tar.xz |
convert: avoid malloc of original file size
We write the output of a "clean" filter into a strbuf. Rather than
growing the strbuf dynamically as we read its output, we make the
initial allocation as large as the original input file. This is a good
guess when the filter is just tweaking a few bytes, but it's disastrous
when the point of the filter is to condense a very large file into a
short identifier (e.g., the way git-lfs and git-annex do). We may ask to
allocate many gigabytes, causing the allocation to fail and Git to
die().
Instead, let's just let strbuf do its usual growth.
When the clean filter does output something around the same size as the
worktree file, the buffer will need to be reallocated until it fits,
starting at 8192 and doubling in size. Benchmarking indicates that
reallocation is not a significant overhead for outputs up to a
few MB in size.
Signed-off-by: Joey Hess <id@joeyh.name>
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/verify-tag.c')
0 files changed, 0 insertions, 0 deletions