]> git.ipfire.org Git - thirdparty/git.git/commit
revision: avoid leak when preparing bloom filter for "/"
authorJeff King <peff@peff.net>
Tue, 4 Aug 2020 07:50:17 +0000 (03:50 -0400)
committerJunio C Hamano <gitster@pobox.com>
Tue, 4 Aug 2020 16:31:57 +0000 (09:31 -0700)
commit398e659e1ec60501d67a0f3cb1a1052c6e50038c
tree4ae93fef031b805958b265679029177f547c3908
parentfd9a631c56ff326bea2956b675f205cd474def4e
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>
revision.c