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

[PATCH v2 2/4] add-interactive: plug a leak in get_untracked_files

From
Rubén Justo <rjusto@gmail.com>
Date
Apr 22, 2024, 22:54 UTC
Message-ID
<791e97af-f233-4c06-9c5b-2b9ff758736e@gmail.com>
In-Reply-To
<69f86760-96ff-4c62-9649-4627652b7c19@gmail.com>

Plug a leak we have since ab1e1cccaf (built-in add -i: re-implement `add-untracked` in C, 2019-11-29).

This leak can be triggered with:
	$ echo a | git add -i

As a curiosity, we have a somewhat similar function in builtin/stash.c, which correctly frees the memory.

Signed-off-by: Rubén Justo <rjusto@gmail.com>
---
 add-interactive.c | 1 +
 1 file changed, 1 insertion(+)
diff --git a/add-interactive.c b/add-interactive.c
index 6bf87e7ae7..e17602b5e4 100644
--- a/add-interactive.c
+++ b/add-interactive.c
@@ -865,6 +865,7 @@ static int get_untracked_files(struct repository *r,
 	}
 
 	strbuf_release(&buf);
+	dir_clear(&dir);
 	return 0;
 }
 
-- 
2.45.0.rc0.4.gfff7e48949
Previous: Rubén JustoNext: Rubén Justo
Message 16 of 19 in “mark t3701-add-interactive.sh as leak-free”
  1. 0/4 mark t3701-add-interactive.sh as leak-freeRubén Justo, Apr 21, 2024
  2. 2/4 add-interactive: plug a leak in get_untracked_filesRubén Justo, Apr 21, 2024
  3. Junio C HamanoApr 22, 2024
  4. 1/4 apply: plug a leak in apply_dataRubén Justo, Apr 21, 2024
  5. Phillip WoodApr 22, 2024
  6. Rubén JustoApr 22, 2024
  7. Junio C HamanoApr 22, 2024
  8. 3/4 add-patch: plug a leak handling the '/' commandRubén Justo, Apr 21, 2024
  9. 4/4 add: plug a leak on interactive_addRubén Justo, Apr 21, 2024
  10. Phillip WoodApr 22, 2024
  11. Rubén JustoApr 22, 2024
  12. Patrick SteinhardtApr 22, 2024
  13. Junio C HamanoApr 22, 2024
  14. 0/4 mark t3701-add-interactive.sh as leak-freeRubén Justo, Apr 22, 2024
  15. 1/4 apply: plug a leak in apply_dataRubén Justo, Apr 22, 2024
  16. 2/4 add-interactive: plug a leak in get_untracked_filesRubén Justo, Apr 22, 2024
  17. 3/4 add-patch: plug a leak handling the '/' commandRubén Justo, Apr 22, 2024
  18. 4/4 add: plug a leak on interactive_addRubén Justo, Apr 22, 2024
  19. Junio C HamanoApr 22, 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.