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 /builtin/pull.c | |
| 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 'builtin/pull.c')
0 files changed, 0 insertions, 0 deletions