diff mbox series

[v2,10/24] submodule--helper: fix "errmsg_str" memory leak

Message ID	patch-v2-10.24-c81a4f69179-20220719T204458Z-avarab@gmail.com (mailing list archive)
State	Superseded
Headers	show Return-Path: <git-owner@kernel.org> From: =?utf-8?b?w4Z2YXIgQXJuZmrDtnLDsCBCamFybWFzb24=?= <avarab@gmail.com> To: git@vger.kernel.org Cc: Junio C Hamano <gitster@pobox.com>, Glen Choo <chooglen@google.com>, Atharva Raykar <raykar.ath@gmail.com>, Prathamesh Chavan <pc44800@gmail.com>, =?utf-8?b?w4Z2YXIgQXJuZmrDtnLDsCBCamFybWFzb24=?= <avarab@gmail.com> Subject: [PATCH v2 10/24] submodule--helper: fix "errmsg_str" memory leak Date: Tue, 19 Jul 2022 22:47:01 +0200 Message-Id: <patch-v2-10.24-c81a4f69179-20220719T204458Z-avarab@gmail.com> In-Reply-To: <cover-v2-00.24-00000000000-20220719T204458Z-avarab@gmail.com> References: <cover-00.11-00000000000-20220713T131601Z-avarab@gmail.com> <cover-v2-00.24-00000000000-20220719T204458Z-avarab@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	submodule--helper: fix memory leaks \| expand [v2,00/24] submodule--helper: fix memory leaks [v2,01/24] submodule--helper: replace memset() with { 0 }-initialization [v2,02/24] submodule--helper: fix a leak in "clone_submodule" [v2,03/24] submodule--helper: fix trivial get_default_remote_submodule() leak [v2,04/24] submodule--helper: fix most "struct pathspec" memory leaks [v2,05/24] submodule--helper: "struct pathspec" memory leak in module_update() [v2,06/24] submodule--helper: don't leak {run,capture}_command() cp.dir argument [v2,07/24] submodule--helper: add "const" to copy of "update_data" [v2,08/24] submodule--helper: add and use *_release() functions [v2,09/24] submodule--helper: refactor "errmsg_str" to be a "struct strbuf" [v2,10/24] submodule--helper: fix "errmsg_str" memory leak [v2,11/24] submodule--helper: fix "sm_path" and other "module_cb_list" leaks [v2,12/24] submodule--helper: fix a leak with repo_clear() [v2,13/24] submodule--helper: fix a memory leak in get_default_remote_submodule() [v2,14/24] submodule--helper: fix "reference" leak is "module_clone_data" [v2,15/24] submodule--helper: fix obscure leak in module_add() [v2,16/24] submodule--helper: fix a leak in module_add() [v2,17/24] submodule--helper: fix a memory leak in print_status() [v2,18/24] submodule--helper: free some "displaypath" in "struct update_data" [v2,19/24] submodule--helper: rename "int res" to "int ret" [v2,20/24] submodule--helper: add skeleton "goto cleanup" to update_submodule() [v2,21/24] submodule--helper: don't exit() on failure, return [v2,22/24] submodule--helper: free rest of "displaypath" in "struct update_data" [v2,23/24] submodule--helper: fix bad config API usage [v2,24/24] submodule--helper: fix a configure_added_submodule() leak

Commit Message

Ævar Arnfjörð Bjarmason July 19, 2022, 8:47 p.m. UTC

Fix a memory leak introduced in e83e3333b57 (submodule: port submodule
subcommand 'summary' from shell to C, 2020-08-13), we sometimes append
to the "errmsg", and need to free the "struct strbuf".

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
 builtin/submodule--helper.c | 1 +
 1 file changed, 1 insertion(+)

diff mbox series

Patch

diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
index c5127c1d50e..e51640d020c 100644
--- a/builtin/submodule--helper.c
+++ b/builtin/submodule--helper.c
@@ -1052,6 +1052,7 @@  static void generate_submodule_summary(struct summary_cb *info,
 	free(displaypath);
 	free(src_abbrev);
 	free(dst_abbrev);
+	strbuf_release(&errmsg);
 }
 
 static void prepare_submodule_summary(struct summary_cb *info,