diff options
author | Jeff King <peff@peff.net> | 2020-08-04 03:50:17 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2020-08-04 09:31:57 -0700 |
commit | 398e659e1ec60501d67a0f3cb1a1052c6e50038c (patch) | |
tree | 4ae93fef031b805958b265679029177f547c3908 /t/t0020-crlf.sh | |
parent | revision: avoid out-of-bounds read/write on empty pathspec (diff) | |
download | tgif-398e659e1ec60501d67a0f3cb1a1052c6e50038c.tar.xz |
revision: avoid leak when preparing bloom filter for "/"
If we're given an empty pathspec, we refuse to set up bloom filters, as
described in f3c2a36810 (revision: empty pathspecs should not use Bloom
filters, 2020-07-01).
But before the empty string check, we drop any trailing slash by
allocating a new string without it. So a pathspec consisting only of "/"
will allocate that string, but then still cause us to bail, leaking the
new string. Let's make sure to free it.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t0020-crlf.sh')
0 files changed, 0 insertions, 0 deletions