On 11-jun-2023 20:29:09, Rubén Justo wrote:
Sorry. This should have been:
[PATCH 00/11] tests: mark as passing with SANITIZE=leak
I'll resend with the correct subject, but will wait some time for
reviews.
Show 65 quoted lines
> The goal in this series is to pass t3200 with SANITIZE=leak.
>
> As a result of the fixes, other tests also pass.
>
> This is the list of tests that no longer trigger any leak after this
> series:
>
> + t1507-rev-parse-upstream.sh
> + t1508-at-combinations.sh
> + t1514-rev-parse-push.sh
> + t2027-checkout-track.sh
> + t3200-branch.sh
> + t3204-branch-name-interpretation.sh
> + t5404-tracking-branches.sh
> + t5517-push-mirror.sh
> + t5525-fetch-tagopt.sh
> + t6040-tracking-info.sh
> + t7508-status.sh
>
> Each of the commits (except 11/11) fixes a leak. They have no
> dependencies on each other. As a result, they can be reordered.
>
> To review one leak, the commit can be moved to the tip or reverted.
> E.g. to review: "branch: fix a leak in check_tracking_branch", this can
> be used:
>
> $ git revert --no-edit HEAD~3
> $ make SANITIZE=leak test T=t3200-branch.sh
>
> Also, each commit have a minimal script in the message that can be used
> to reproduce the leak.
>
> Rubén Justo (11):
> rev-parse: fix a leak with --abbrev-ref
> config: fix a leak in git_config_copy_or_rename_section_in_file
> remote: fix a leak in query_matches_negative_refspec
> branch: fix a leak in dwim_and_setup_tracking
> branch: fix a leak in setup_tracking
> branch: fix a leak in cmd_branch
> branch: fix a leak in inherit_tracking
> branch: fix a leak in check_tracking_branch
> branch: fix a leak in setup_tracking
> config: fix a leak in git_config_copy_or_rename_section_in_file
> tests: mark as passing with SANITIZE=leak
>
> branch.c | 14 +++++++++-----
> builtin/branch.c | 2 +-
> builtin/rev-parse.c | 7 +++++--
> config.c | 2 ++
> remote.c | 2 +-
> t/t1507-rev-parse-upstream.sh | 1 +
> t/t1508-at-combinations.sh | 1 +
> t/t1514-rev-parse-push.sh | 1 +
> t/t2027-checkout-track.sh | 1 +
> t/t3200-branch.sh | 1 +
> t/t3204-branch-name-interpretation.sh | 1 +
> t/t5404-tracking-branches.sh | 1 +
> t/t5517-push-mirror.sh | 1 +
> t/t5525-fetch-tagopt.sh | 1 +
> t/t6040-tracking-info.sh | 1 +
> t/t7508-status.sh | 1 +
> 16 files changed, 29 insertions(+), 9 deletions(-)
>
> --
> 2.40.1
Rubén Justo <rjusto@gmail.com>