[PATCH v2 0/3] setup: add `clear_repository_format()`
- From
- Martin Ågren <martin.agren@gmail.com>
- Date
- Jan 14, 2019, 18:34 UTC
- Message-ID
- <cover.1547488709.git.martin.agren@gmail.com>
- In-Reply-To
- <20181218072528.3870492-1-martin.agren@gmail.com>
This patch series addresses memory leaks related to `struct repository_format`. Compared to v1 [1], this v2 has dropped the first patch ("setup: drop return value from `read_repository_format()`") and added another patch 1/3 (which solves a different problem).
Patch 2/3 is unchanged. Patch 3/3 now calls `clear_...()` not only in the error paths, but in all paths, as suggested in the review of v1.
Thanks to Peff and brian for reviewing v1.
There's a minor textual conflict with ed/simplify-setup-git-dir. A merge of these two topics still passes the test suite (and these leaks remain plugged).
[1] https://public-inbox.org/git/20181218072528.3870492-1-martin.agren@gmail.com/#t
Martin Ågren (3): setup: free old value before setting `work_tree` setup: do not use invalid `repository_format` setup: add `clear_repository_format()`
cache.h | 12 ++++++++++++ builtin/init-db.c | 3 ++- repository.c | 3 ++- setup.c | 33 ++++++++++++++++++++++++--------- worktree.c | 4 +++- 5 files changed, 43 insertions(+), 12 deletions(-)
-- 2.20.1