Patrick Steinhardt c92abe71df builtin/fetch: fix leaking transaction with --atomic ...

With the `--atomic` flag, we use a single ref transaction to commit all
ref updates in git-fetch(1). The lifetime of transactions is somewhat
weird: while `ref_transaction_abort()` will free the transaction, a call
to `ref_transaction_commit()` won't. We thus have to manually free the
transaction in the successful case.

Adapt the code to free the transaction in the exit path to plug the
resulting memory leak. As `ref_transaction_abort()` already freed the
transaction for us, we have to unset the transaction when we hit that
code path to not cause a double free.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

2024-08-22 09:18:06 -07:00

..

add.c

add-i: finally retire add.interactive.useBuiltin

2024-06-05 14:53:26 -07:00

am.c

merge-recursive: honor diff.algorithm

2024-07-13 18:10:49 -07:00

annotate.c

…

apply.c

apply: fix uninitialized hash function

2024-05-21 09:07:48 -07:00

archive.c

builtin/archive: fix leaking OPT_FILENAME() value

2024-08-22 09:18:04 -07:00

bisect.c

refs: add referent to each_ref_fn

2024-08-09 08:47:34 -07:00

blame.c

Merge branch 'ps/leakfixes-more'

2024-07-08 14:53:10 -07:00

branch.c

Merge branch 'kn/ref-transaction-symref'

2024-05-20 11:20:04 -07:00

bugreport.c

global: improve const correctness when assigning string constants

2024-06-07 10:30:48 -07:00

bundle.c

builtin/bundle: abort "verify" early when there is no repository

2024-05-06 22:50:49 -07:00

cat-file.c

object-name: free leaking object contexts

2024-06-11 13:15:05 -07:00

check-attr.c

builtin: stop using the_index

2024-04-18 12:30:42 -07:00

check-ignore.c

global: improve const correctness when assigning string constants

2024-06-07 10:30:48 -07:00

check-mailmap.c

builtin.h: remove unneccessary includes

2023-06-21 13:39:54 -07:00

check-ref-format.c

cache.h: remove this no-longer-used header

2023-06-21 13:39:53 -07:00

checkout--worker.c

git-compat-util: move alloc macros to git-compat-util.h

2023-07-05 11:42:31 -07:00

checkout-index.c

builtin: stop using the_index

2024-04-18 12:30:42 -07:00

checkout.c

Merge branch 'jc/refs-symref-referent'

2024-08-15 13:22:15 -07:00

clean.c

builtin: stop using the_index

2024-04-18 12:30:42 -07:00

clone.c

builtin/clone: propagate ref storage format to submodules

2024-08-08 09:21:39 -07:00

column.c

column: disallow negative padding

2024-02-13 10:18:50 -08:00

commit-graph.c

Merge branch 'ja/doc-placeholders-fix'

2024-02-08 13:20:34 -08:00

commit-tree.c

commit: fix leaking parents when calling commit_tree_extended()

2024-06-11 13:15:07 -07:00

commit.c

config: fix leaking comment character config

2024-08-14 10:07:58 -07:00

config.c

Merge branch 'ps/leakfixes'

2024-06-06 12:49:23 -07:00

count-objects.c

count-objects: mark unused parameter in alternates callback

2023-07-13 17:24:00 -07:00

credential-cache--daemon.c

Merge branch 'bc/credential-scheme-enhancement'

2024-05-08 10:18:44 -07:00

credential-cache.c

builtin/credential-cache: fix trivial leaks

2024-08-01 08:47:37 -07:00

credential-store.c

builtin/credential-store: fix leaking credential

2024-08-01 08:47:36 -07:00

credential.c

builtin/credential: clear credential before exit

2024-05-27 11:20:01 -07:00

describe.c

Merge branch 'jc/refs-symref-referent'

2024-08-15 13:22:15 -07:00

diagnose.c

global: improve const correctness when assigning string constants

2024-06-07 10:30:48 -07:00

diff-files.c

treewide: remove unnecessary includes in source files

2023-12-26 12:04:31 -08:00

diff-index.c

treewide: remove unnecessary includes in source files

2023-12-26 12:04:31 -08:00

diff-tree.c

Merge branch 'xx/diff-tree-remerge-diff-fix'

2024-08-15 13:22:16 -07:00

diff.c

builtin/diff: free symmetric diff members

2024-08-14 10:08:02 -07:00

difftool.c

Merge branch 'ps/leakfixes-more'

2024-07-08 14:53:10 -07:00

fast-export.c

builtin/fast-export: plug leaking tag names

2024-08-14 10:07:59 -07:00

fast-import.c

builtin/fast-import: plug trivial memory leaks

2024-08-14 10:07:59 -07:00

fetch-pack.c

builtin/fetch-pack: fix leaking refs

2024-08-22 09:18:05 -07:00

fetch.c

builtin/fetch: fix leaking transaction with --atomic

2024-08-22 09:18:06 -07:00

fmt-merge-msg.c

parse-options: fix leaks for users of OPT_FILENAME

2024-06-11 13:15:04 -07:00

for-each-ref.c

ref-filter: properly distinuish pseudo and root refs

2024-05-15 07:30:52 -07:00

for-each-repo.c

for-each-repo: optionally keep going on an error

2024-04-24 10:46:03 -07:00

fsck.c

Merge branch 'sj/ref-fsck'

2024-08-16 12:51:51 -07:00

fsmonitor--daemon.c

treewide: remove unnecessary includes in source files

2023-12-26 12:04:33 -08:00

gc.c

refs: add referent to each_ref_fn

2024-08-09 08:47:34 -07:00

get-tar-commit-id.c

set errno=0 before strtoX calls

2024-08-05 10:59:20 -07:00

grep.c

object-name: free leaking object contexts

2024-06-11 13:15:05 -07:00

hash-object.c

builtin/hash-object: fix uninitialized hash function

2024-05-21 09:05:13 -07:00

help.c

Merge branch 'gc/config-context'

2023-07-06 11:54:48 -07:00

hook.c

treewide: remove unnecessary includes in source files

2023-12-26 12:04:31 -08:00

index-pack.c

hash: require hash algorithm in oidread() and oidclr()

2024-06-14 10:26:32 -07:00

init-db.c

refs: convert ref storage format to an enum

2024-06-06 09:04:31 -07:00

interpret-trailers.c

Merge branch 'la/hide-trailer-info'

2024-05-23 11:04:27 -07:00

log.c

builtin/log: fix leak when showing converted blob contents

2024-08-14 10:08:01 -07:00

ls-files.c

factor out strbuf_expand_bad_format()

2024-03-25 11:59:24 -07:00

ls-remote.c

Merge branch 'ps/ls-remote-out-of-repo-fix'

2024-08-14 14:54:49 -07:00

ls-tree.c

object-name: free leaking object contexts

2024-06-11 13:15:05 -07:00

mailinfo.c

treewide: remove unnecessary includes in source files

2023-12-26 12:04:31 -08:00

mailsplit.c

global: improve const correctness when assigning string constants

2024-06-07 10:30:48 -07:00

merge-base.c

commit-reach(repo_get_merge_bases_many_dirty): pass on errors

2024-02-29 08:06:01 -08:00

merge-file.c

merge-file: add --diff-algorithm option

2023-11-22 14:23:06 +09:00

merge-index.c

builtin: stop using the_index

2024-04-18 12:30:42 -07:00

merge-ours.c

diff.h: remove unnecessary include of oidset.h

2023-06-21 13:39:53 -07:00

merge-recursive.c

merge-recursive: honor diff.algorithm

2024-07-13 18:10:49 -07:00

merge-tree.c

builtin/merge-tree: fix leaking -X strategy options

2024-08-22 09:18:04 -07:00

merge.c

merge-recursive: honor diff.algorithm

2024-07-13 18:10:49 -07:00

mktag.c

fsck: make "fsck_error" callback generic

2024-08-08 09:36:52 -07:00

mktree.c

git-compat-util: move alloc macros to git-compat-util.h

2023-07-05 11:42:31 -07:00

multi-pack-index.c

midx: implement support for writing incremental MIDX chains

2024-08-06 12:01:39 -07:00

mv.c

mv: replace src_dir with a strvec

2024-05-30 08:55:29 -07:00

name-rev.c

Merge branch 'jc/refs-symref-referent'

2024-08-15 13:22:15 -07:00

notes.c

Merge branch 'ps/leakfixes-part-4' into ps/leakfixes-part-5

2024-08-20 10:15:27 -07:00

pack-objects.c

refs: add referent to each_ref_fn

2024-08-09 08:47:34 -07:00

pack-redundant.c

hash: require hash algorithm in oidread() and oidclr()

2024-06-14 10:26:32 -07:00

pack-refs.c

builtin/pack-refs: introduce new "--auto" flag

2024-03-25 09:54:07 -07:00

patch-id.c

Merge branch 'jc/patch-id'

2024-08-14 14:54:53 -07:00

prune-packed.c

treewide: be explicit about dependence on gettext.h

2023-03-21 10:56:51 -07:00

prune.c

Merge branch 'en/header-split-cache-h-part-3'

2023-06-29 16:43:21 -07:00

pull.c

Merge branch 'ps/use-the-repository'

2024-07-02 09:59:00 -07:00

push.c

builtin/push: call set_refspecs after validating remote

2024-07-12 09:14:11 -07:00

range-diff.c

treewide: remove unnecessary includes in source files

2023-12-26 12:04:31 -08:00

read-tree.c

builtin: stop using the_index

2024-04-18 12:30:42 -07:00

rebase.c

builtin/rebase: fix leaking commit.gpgsign value

2024-08-14 10:07:58 -07:00

receive-pack.c

refs: add referent to each_ref_fn

2024-08-09 08:47:34 -07:00

reflog.c

refs: remove dwim_log()

2024-05-17 10:33:39 -07:00

refs.c

builtin/refs: add verify subcommand

2024-08-08 09:36:53 -07:00

remote-ext.c

builtins: annotate always-empty prefix parameters

2023-03-28 14:11:24 -07:00

remote-fd.c

builtins: annotate always-empty prefix parameters

2023-03-28 14:11:24 -07:00

remote.c

Merge branch 'jc/refs-symref-referent'

2024-08-15 13:22:15 -07:00

repack.c

builtin/repack: fix leaks when computing packs to repack

2024-08-22 09:18:04 -07:00

replace.c

refs: add referent to each_ref_fn

2024-08-09 08:47:34 -07:00

replay.c

Merge branch 'ps/leakfixes-part-3'

2024-08-14 14:54:47 -07:00

rerere.c

builtin/rerere: fix various trivial memory leaks

2024-08-01 08:47:37 -07:00

reset.c

Merge branch 'ps/refs-without-the-repository'

2024-05-16 10:10:14 -07:00

rev-list.c

builtin/rev-list: fix leaking bitmap index when calculating disk usage

2024-06-11 13:15:05 -07:00

rev-parse.c

Merge branch 'jc/refs-symref-referent'

2024-08-15 13:22:15 -07:00

revert.c

global: improve const correctness when assigning string constants

2024-06-07 10:30:48 -07:00

rm.c

hash: require hash algorithm in oidread() and oidclr()

2024-06-14 10:26:32 -07:00

send-pack.c

builtin/send-pack: fix leaking refspecs

2024-08-22 09:18:05 -07:00

shortlog.c

builtin/shortlog: fix various trivial memory leaks

2024-08-01 08:47:37 -07:00

show-branch.c

Merge branch 'jc/refs-symref-referent'

2024-08-15 13:22:15 -07:00

show-index.c

cache.h: remove this no-longer-used header

2023-06-21 13:39:53 -07:00

show-ref.c

Merge branch 'jc/refs-symref-referent'

2024-08-15 13:22:15 -07:00

sparse-checkout.c

Merge branch 'ps/leakfixes-more'

2024-07-08 14:53:10 -07:00

stash.c

Merge branch 'ps/leakfixes-part-3'

2024-08-14 14:54:47 -07:00

stripspace.c

strbuf: accept a comment string for strbuf_add_commented_lines()

2024-03-12 13:28:10 -07:00

submodule--helper.c

Merge branch 'jc/refs-symref-referent'

2024-08-15 13:22:15 -07:00

symbolic-ref.c

Merge branch 'kn/ref-transaction-symref'

2024-05-20 11:20:04 -07:00

tag.c

hash: require hash algorithm in oidread() and oidclr()

2024-06-14 10:26:32 -07:00

unpack-file.c

treewide: remove unnecessary includes for wrapper.h

2023-07-05 11:41:59 -07:00

unpack-objects.c

hash: require hash algorithm in oidread() and oidclr()

2024-06-14 10:26:32 -07:00

update-index.c

refs: refactor resolve_gitlink_ref() to accept a repository

2024-05-17 10:33:38 -07:00

update-ref.c

Merge branch 'ps/use-the-repository'

2024-07-02 09:59:00 -07:00

update-server-info.c

cache.h: remove this no-longer-used header

2023-06-21 13:39:53 -07:00

upload-archive.c

builtin/upload-archive: fix leaking args passed to write_archive()

2024-08-22 09:18:04 -07:00

upload-pack.c

Sync with 2.44.1

2024-04-29 20:42:30 +02:00

var.c

var(win32): do report the GIT_SHELL_PATH that is actually used

2024-07-13 16:23:37 -07:00

verify-commit.c

treewide: remove unnecessary includes in source files

2023-12-26 12:04:31 -08:00

verify-pack.c

builtin.h: remove unneccessary includes

2023-06-21 13:39:54 -07:00

verify-tag.c

treewide: remove unnecessary includes in source files

2023-12-26 12:04:31 -08:00

worktree.c

Merge branch 'jc/refs-symref-referent'

2024-08-15 13:22:15 -07:00

write-tree.c

builtin: stop using the_index

2024-04-18 12:30:42 -07:00