Thread (162 messages) 162 messages, 2 authors, 2022-09-01
STALE1368d
Revisions (5)
  1. v2 current
  2. v3 [diff vs current]
  3. v5 [diff vs current]
  4. v6 [diff vs current]
  5. v7 [diff vs current]

[PATCH v2 12/24] submodule--helper: fix a leak with repo_clear()

From: Ævar Arnfjörð Bjarmason <hidden>
Date: 2022-07-19 20:48:00
Subsystem: the rest · Maintainer: Linus Torvalds

Call repo_clear() in ensure_core_worktree() to free the "struct
repository". Fixes a leak that's been here since
74d4731da1f (submodule--helper: replace connect-gitdir-workingtree by
ensure-core-worktree, 2018-08-13).

Signed-off-by: Ævar Arnfjörð Bjarmason <redacted>
---
 builtin/submodule--helper.c | 1 +
 1 file changed, 1 insertion(+)
diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
index a95231b1698..339ba94e658 100644
--- a/builtin/submodule--helper.c
+++ b/builtin/submodule--helper.c
@@ -2457,6 +2457,7 @@ static void ensure_core_worktree(const char *path)
 		free(abs_path);
 		strbuf_release(&sb);
 	}
+	repo_clear(&subrepo);
 }
 
 static const char *submodule_update_type_to_label(enum submodule_update_type type)
-- 
2.37.1.1062.g385eac7fccf
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help