]> git.ipfire.org Git - thirdparty/git.git/commit - t/t1020-subdirectory.sh
log: fix a memory leak in "git show <revision>..."
authorÆvar Arnfjörð Bjarmason <avarab@gmail.com>
Tue, 2 Aug 2022 15:33:13 +0000 (17:33 +0200)
committerJunio C Hamano <gitster@pobox.com>
Wed, 3 Aug 2022 17:16:28 +0000 (10:16 -0700)
commit055e57b7b2183cd5ae7ce8af0becd20623b3db71
tree632110ef8f8288bbf62700700b01b2ff29224896
parentc541e77cf83707d04eeaea4e25f1147017397afe
log: fix a memory leak in "git show <revision>..."

Fix a memory leak in code added in 5d7eeee2ac6 (git-show: grok blobs,
trees and tags, too, 2006-12-14). As we iterate over a "<revision>..."
command-line and encounter ad OBJ_COMMIT we want to use our "struct
rev_info", but with a "pending" array of one element: the one commit
we're showing in the loop.

To do this 5d7eeee2ac6 saved away a pointer to rev.pending.objects and
rev.pending.nr for its iteration. We'd then clobber those (and alloc)
when we needed to show an OBJ_COMMIT.

We'd therefore leak the "rev.pending" we started out with, and only
free the new "rev.pending" in the "OBJ_COMMIT" case arm as
prepare_revision_walk() would draw it down.

Let's fix this memory leak. Now when we encounter an OBJ_COMMIT we
save away the "rev.pending" before clearing it. We then add a single
commit to it, which our indirect invocation of prepare_revision_walk()
will remove. After that we restore the "rev.pending".

Our "rev.pending" will then get free'd by the release_revisions()
added in f6bfea0ad01 (revisions API users: use release_revisions() in
builtin/log.c, 2022-04-13)

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/log.c
t/t0203-gettext-setlocale-sanity.sh
t/t1020-subdirectory.sh
t/t3307-notes-man.sh
t/t3920-crlf-messages.sh
t/t4069-remerge-diff.sh
t/t7007-show.sh
t/t7503-pre-commit-and-pre-merge-commit-hooks.sh
t/t9122-git-svn-author.sh
t/t9162-git-svn-dcommit-interactive.sh