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. > 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
