git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 17/21] t/helper: stop re-initialization of `the_repository`

From
Patrick Steinhardt <ps@pks.im>
Date
Oct 11, 2024, 05:32 UTC
Message-ID
<8e10ee844c6bcc98f1a848452932408b20244fda.1728624670.git.ps@pks.im>
In-Reply-To
<cover.1728624670.git.ps@pks.im>

While "common-main.c" already initializes `the_repository` for us, we do so a second time in the "read-cache" test helper. This causes a memory leak because the old repository's contents isn't released.

Stop calling `initialize_repository()` to plug this leak.
Signed-off-by: Patrick Steinhardt <ps@pks.im>
---
 t/helper/test-read-cache.c  | 2 --
 t/t7519-status-fsmonitor.sh | 1 +
 2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/t/helper/test-read-cache.c b/t/helper/test-read-cache.c
index d285c656bd3..e277dde8e71 100644
--- a/t/helper/test-read-cache.c
+++ b/t/helper/test-read-cache.c
@@ -11,8 +11,6 @@ int cmd__read_cache(int argc, const char **argv)
 	int i, cnt = 1;
 	const char *name = NULL;
 
-	initialize_repository(the_repository);
-
 	if (argc > 1 && skip_prefix(argv[1], "--print-and-refresh=", &name)) {
 		argc--;
 		argv++;
diff --git a/t/t7519-status-fsmonitor.sh b/t/t7519-status-fsmonitor.sh
index 7ee69ecdd4a..0f88a58a819 100755
--- a/t/t7519-status-fsmonitor.sh
+++ b/t/t7519-status-fsmonitor.sh
@@ -2,6 +2,7 @@
 
 test_description='git status with file system watcher'
 
+TEST_PASSES_SANITIZE_LEAK=true
 . ./test-lib.sh
 
 # Note, after "git reset --hard HEAD" no extensions exist other than 'TREE'
-- 
2.47.0.dirty
Previous: Patrick SteinhardtNext: Patrick Steinhardt
Message 23 of 100 in “Memory leak fixes (pt.9)”
  1. 00/21 Memory leak fixes (pt.9)Patrick Steinhardt, Oct 11, 2024
  2. 01/21 builtin/ls-remote: plug leaking server optionsPatrick Steinhardt, Oct 11, 2024
  3. 02/21 t/helper: fix leaks in "reach" test toolPatrick Steinhardt, Oct 11, 2024
  4. 03/21 grep: fix leak in `grep_splice_or()`Patrick Steinhardt, Oct 11, 2024
  5. 04/21 builtin/grep: fix leak with `--max-count=0`Patrick Steinhardt, Oct 11, 2024
  6. 05/21 revision: fix leaking bloom filtersPatrick Steinhardt, Oct 11, 2024
  7. 06/21 diff-lib: fix leaking diffopts in `do_diff_cache()`Patrick Steinhardt, Oct 11, 2024
  8. Kristoffer HaugsbakkOct 21, 2024
  9. 07/21 pretty: clear signature checkPatrick Steinhardt, Oct 11, 2024
  10. Toon ClaesOct 18, 2024
  11. Patrick SteinhardtOct 21, 2024
  12. 08/21 upload-pack: fix leaking URI protocolsPatrick Steinhardt, Oct 11, 2024
  13. 09/21 builtin/commit: fix leaking change data contentsPatrick Steinhardt, Oct 11, 2024
  14. 10/21 trailer: fix leaking trailer valuesPatrick Steinhardt, Oct 11, 2024
  15. Toon ClaesOct 18, 2024
  16. Patrick SteinhardtOct 21, 2024
  17. 11/21 builtin/commit: fix leaking cleanup configPatrick Steinhardt, Oct 11, 2024
  18. 12/21 transport-helper: fix leaking import/export marksPatrick Steinhardt, Oct 11, 2024
  19. 13/21 builtin/tag: fix leaking key ID on failure to signPatrick Steinhardt, Oct 11, 2024
  20. 14/21 combine-diff: fix leaking lost linesPatrick Steinhardt, Oct 11, 2024
  21. 15/21 dir: release untracked cache dataPatrick Steinhardt, Oct 11, 2024
  22. 16/21 sparse-index: correctly free EWAH contentsPatrick Steinhardt, Oct 11, 2024
  23. 17/21 t/helper: stop re-initialization of `the_repository`Patrick Steinhardt, Oct 11, 2024
  24. 18/21 t/helper: fix leaking buffer in "dump-untracked-cache"Patrick Steinhardt, Oct 11, 2024
  25. 19/21 dir: fix leak when parsing "status.showUntrackedFiles"Patrick Steinhardt, Oct 11, 2024
  26. 20/21 builtin/merge: release outbut buffer after performing mergePatrick Steinhardt, Oct 11, 2024
  27. Toon ClaesOct 18, 2024
  28. Patrick SteinhardtOct 21, 2024
  29. 21/21 list-objects-filter-options: work around reported leak on errorPatrick Steinhardt, Oct 11, 2024
  30. Toon ClaesOct 18, 2024
  31. Patrick SteinhardtOct 21, 2024
  32. Taylor BlauOct 18, 2024
  33. Patrick SteinhardtOct 21, 2024
  34. 00/22 Memory leak fixes (pt.9)Patrick Steinhardt, Oct 21, 2024
  35. 01/22 builtin/ls-remote: plug leaking server optionsPatrick Steinhardt, Oct 21, 2024
  36. Justin ToblerNov 4, 2024
  37. Kristoffer HaugsbakkNov 4, 2024
  38. 02/22 t/helper: fix leaks in "reach" test toolPatrick Steinhardt, Oct 21, 2024
  39. 03/22 grep: fix leak in `grep_splice_or()`Patrick Steinhardt, Oct 21, 2024
  40. Kristoffer HaugsbakkOct 21, 2024
  41. 04/22 builtin/grep: fix leak with `--max-count=0`Patrick Steinhardt, Oct 21, 2024
  42. 05/22 revision: fix leaking bloom filtersPatrick Steinhardt, Oct 21, 2024
  43. 06/22 diff-lib: fix leaking diffopts in `do_diff_cache()`Patrick Steinhardt, Oct 21, 2024
  44. 07/22 pretty: clear signature checkPatrick Steinhardt, Oct 21, 2024
  45. Justin ToblerNov 4, 2024
  46. 08/22 upload-pack: fix leaking URI protocolsPatrick Steinhardt, Oct 21, 2024
  47. 09/22 builtin/commit: fix leaking change data contentsPatrick Steinhardt, Oct 21, 2024
  48. Justin ToblerNov 4, 2024
  49. 10/22 trailer: fix leaking trailer valuesPatrick Steinhardt, Oct 21, 2024
  50. Justin ToblerNov 4, 2024
  51. Patrick SteinhardtNov 5, 2024
  52. 11/22 trailer: fix leaking strbufs when formatting trailersPatrick Steinhardt, Oct 21, 2024
  53. Taylor BlauOct 21, 2024
  54. Justin ToblerNov 4, 2024
  55. Patrick SteinhardtNov 5, 2024
  56. 12/22 builtin/commit: fix leaking cleanup configPatrick Steinhardt, Oct 21, 2024
  57. 13/22 transport-helper: fix leaking import/export marksPatrick Steinhardt, Oct 21, 2024
  58. 14/22 builtin/tag: fix leaking key ID on failure to signPatrick Steinhardt, Oct 21, 2024
  59. 15/22 combine-diff: fix leaking lost linesPatrick Steinhardt, Oct 21, 2024
  60. Justin ToblerNov 4, 2024
  61. Patrick SteinhardtNov 5, 2024
  62. 16/22 dir: release untracked cache dataPatrick Steinhardt, Oct 21, 2024
  63. 17/22 sparse-index: correctly free EWAH contentsPatrick Steinhardt, Oct 21, 2024
  64. 18/22 t/helper: stop re-initialization of `the_repository`Patrick Steinhardt, Oct 21, 2024
  65. 19/22 t/helper: fix leaking buffer in "dump-untracked-cache"Patrick Steinhardt, Oct 21, 2024
  66. 20/22 dir: fix leak when parsing "status.showUntrackedFiles"Patrick Steinhardt, Oct 21, 2024
  67. 21/22 builtin/merge: release outbut buffer after performing mergePatrick Steinhardt, Oct 21, 2024
  68. Kristoffer HaugsbakkOct 21, 2024
  69. 22/22 list-objects-filter-options: work around reported leak on errorPatrick Steinhardt, Oct 21, 2024
  70. Kristoffer HaugsbakkOct 21, 2024
  71. Patrick SteinhardtOct 21, 2024
  72. Toon ClaesOct 25, 2024
  73. Justin ToblerNov 4, 2024
  74. Patrick SteinhardtNov 5, 2024
  75. 00/22 Memory leak fixes (pt.9)Patrick Steinhardt, Nov 5, 2024
  76. 01/22 builtin/ls-remote: plug leaking server optionsPatrick Steinhardt, Nov 5, 2024
  77. 02/22 t/helper: fix leaks in "reach" test toolPatrick Steinhardt, Nov 5, 2024
  78. 03/22 grep: fix leak in `grep_splice_or()`Patrick Steinhardt, Nov 5, 2024
  79. 04/22 builtin/grep: fix leak with `--max-count=0`Patrick Steinhardt, Nov 5, 2024
  80. 05/22 revision: fix leaking bloom filtersPatrick Steinhardt, Nov 5, 2024
  81. 06/22 diff-lib: fix leaking diffopts in `do_diff_cache()`Patrick Steinhardt, Nov 5, 2024
  82. 07/22 pretty: clear signature checkPatrick Steinhardt, Nov 5, 2024
  83. 08/22 upload-pack: fix leaking URI protocolsPatrick Steinhardt, Nov 5, 2024
  84. 09/22 builtin/commit: fix leaking change data contentsPatrick Steinhardt, Nov 5, 2024
  85. 10/22 trailer: fix leaking trailer valuesPatrick Steinhardt, Nov 5, 2024
  86. 11/22 trailer: fix leaking strbufs when formatting trailersPatrick Steinhardt, Nov 5, 2024
  87. 12/22 builtin/commit: fix leaking cleanup configPatrick Steinhardt, Nov 5, 2024
  88. 13/22 transport-helper: fix leaking import/export marksPatrick Steinhardt, Nov 5, 2024
  89. 14/22 builtin/tag: fix leaking key ID on failure to signPatrick Steinhardt, Nov 5, 2024
  90. 15/22 combine-diff: fix leaking lost linesPatrick Steinhardt, Nov 5, 2024
  91. 16/22 dir: release untracked cache dataPatrick Steinhardt, Nov 5, 2024
  92. 17/22 sparse-index: correctly free EWAH contentsPatrick Steinhardt, Nov 5, 2024
  93. 18/22 t/helper: stop re-initialization of `the_repository`Patrick Steinhardt, Nov 5, 2024
  94. 19/22 t/helper: fix leaking buffer in "dump-untracked-cache"Patrick Steinhardt, Nov 5, 2024
  95. 20/22 dir: fix leak when parsing "status.showUntrackedFiles"Patrick Steinhardt, Nov 5, 2024
  96. 21/22 builtin/merge: release output buffer after performing mergePatrick Steinhardt, Nov 5, 2024
  97. 22/22 list-objects-filter-options: work around reported leak on errorPatrick Steinhardt, Nov 5, 2024
  98. Junio C HamanoNov 5, 2024
  99. Patrick SteinhardtNov 5, 2024
  100. Justin ToblerNov 5, 2024

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.