[v3,24/27] t: mark some tests as leak free

Message ID	20241120-b4-pks-leak-fixes-pt10-v3-24-d67f08f45c74@pks.im (mailing list archive)
State	New
Headers	show Received: from fhigh-a2-smtp.messagingengine.com (fhigh-a2-smtp.messagingengine.com [103.168.172.153]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 943AD1C1F26 for <git@vger.kernel.org>; Wed, 20 Nov 2024 13:40:12 +0000 (UTC) Feedback-ID: i197146af:Fastmail From: Patrick Steinhardt <ps@pks.im> Date: Wed, 20 Nov 2024 14:39:53 +0100 Subject: [PATCH v3 24/27] t: mark some tests as leak free Precedence: bulk MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20241120-b4-pks-leak-fixes-pt10-v3-24-d67f08f45c74@pks.im> References: <20241120-b4-pks-leak-fixes-pt10-v3-0-d67f08f45c74@pks.im> In-Reply-To: <20241120-b4-pks-leak-fixes-pt10-v3-0-d67f08f45c74@pks.im> To: git@vger.kernel.org Cc: =?utf-8?q?Rub=C3=A9n_Justo?= <rjusto@gmail.com>, Jeff King <peff@peff.net>, Toon Claes <toon@iotcl.com>
Series	Memory leak fixes (pt.10, final) \| expand [v3,00/27] Memory leak fixes (pt.10, final) [v3,01/27] builtin/blame: fix leaking blame entries with `--incremental` [v3,02/27] bisect: fix leaking good/bad terms when reading multipe times [v3,03/27] bisect: fix leaking string in `handle_bad_merge_base()` [v3,04/27] bisect: fix leaking `current_bad_oid` [v3,05/27] bisect: fix multiple leaks in `bisect_next_all()` [v3,06/27] bisect: fix leaking commit list items in `check_merge_base()` [v3,07/27] bisect: fix various cases where we leak commit list items [v3,08/27] line-log: fix leak when rewriting commit parents [v3,09/27] strvec: introduce new `strvec_splice()` function [v3,10/27] git: refactor alias handling to use a `struct strvec` [v3,11/27] git: refactor builtin handling to use a `struct strvec` [v3,12/27] split-index: fix memory leak in `move_cache_to_base_index()` [v3,13/27] builtin/sparse-checkout: fix leaking sanitized patterns [v3,14/27] help: refactor to not use globals for reading config [v3,15/27] help: fix leaking `struct cmdnames` [v3,16/27] help: fix leaking return value from `help_unknown_cmd()` [v3,17/27] builtin/help: fix leaks in `check_git_cmd()` [v3,18/27] builtin/init-db: fix leaking directory paths [v3,19/27] builtin/branch: fix leaking sorting options [v3,20/27] t/helper: fix leaking commit graph in "read-graph" subcommand [v3,21/27] global: drop `UNLEAK()` annotation [v3,22/27] git-compat-util: drop now-unused `UNLEAK()` macro [v3,23/27] t5601: work around leak sanitizer issue [v3,24/27] t: mark some tests as leak free [v3,25/27] t: remove unneeded !SANITIZE_LEAK prerequisites [v3,26/27] test-lib: unconditionally enable leak checking [v3,27/27] t: remove TEST_PASSES_SANITIZE_LEAK annotations

Message ID

20241120-b4-pks-leak-fixes-pt10-v3-24-d67f08f45c74@pks.im (mailing list archive)

State

New

Headers

Feedback-ID: i197146af:Fastmail
From: Patrick Steinhardt <ps@pks.im>
Date: Wed, 20 Nov 2024 14:39:53 +0100
Subject: [PATCH v3 24/27] t: mark some tests as leak free
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Message-Id: <20241120-b4-pks-leak-fixes-pt10-v3-24-d67f08f45c74@pks.im>
References: <20241120-b4-pks-leak-fixes-pt10-v3-0-d67f08f45c74@pks.im>
In-Reply-To: <20241120-b4-pks-leak-fixes-pt10-v3-0-d67f08f45c74@pks.im>
To: git@vger.kernel.org
Cc: =?utf-8?q?Rub=C3=A9n_Justo?= <rjusto@gmail.com>,
  Jeff King <peff@peff.net>, Toon Claes <toon@iotcl.com>

Series

Memory leak fixes (pt.10, final) | expand

Commit Message

Patrick Steinhardt Nov. 20, 2024, 1:39 p.m. UTC

Both t5558 and t5601 are leak-free starting with 6dab49b9fb (bundle-uri:
plug leak in unbundle_from_file(), 2024-10-10). Mark them accordingly.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
---
 t/t5558-clone-bundle-uri.sh | 1 +
 t/t5601-clone.sh            | 1 +
 2 files changed, 2 insertions(+)

diff --git a/t/t5558-clone-bundle-uri.sh b/t/t5558-clone-bundle-uri.sh
index 3816ed5058d90118b4cbc700d65e97aa9c9987c0..b488b505b928ab6e58258360364ba53bec704eca 100755
--- a/t/t5558-clone-bundle-uri.sh
+++ b/t/t5558-clone-bundle-uri.sh
@@ -2,6 +2,7 @@ 
 
 test_description='test fetching bundles with --bundle-uri'
 
+TEST_PASSES_SANITIZE_LEAK=true
 . ./test-lib.sh
 . "$TEST_DIRECTORY"/lib-bundle.sh
 
diff --git a/t/t5601-clone.sh b/t/t5601-clone.sh
index d0c18660e33113e64a293c695239a1c1a7975add..3b5285cfcf326fa02015d7a808b4312c62283487 100755
--- a/t/t5601-clone.sh
+++ b/t/t5601-clone.sh
@@ -5,6 +5,7 @@  test_description=clone
 GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
 export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
 
+TEST_PASSES_SANITIZE_LEAK=true
 . ./test-lib.sh
 
 X=

[v3,24/27] t: mark some tests as leak free

Commit Message

Patch