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

[Outreachy] [PATCH] bisect--helper: refer branch.buf before strbuf_release(...)

From
Miriam Rubio <mirucam@gmail.com>
Date
Dec 8, 2019, 17:28 UTC
Message-ID
<20191208172813.16518-1-mirucam@gmail.com>
From: Tanushree Tumane <tanushreetumane@gmail.com>

Move `error("...%s...", branch.buf);` before `strbuf_release(&branch);` to release string buffer `branch` and the memory it used.

Mentored-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Mentored-by: Christian Couder <chriscool@tuxfamily.org>
Signed-off-by: Tanushree Tumane <tanushreetumane@gmail.com>
Signed-off-by: Miriam Rubio <mirucam@gmail.com>
---
 builtin/bisect--helper.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/builtin/bisect--helper.c b/builtin/bisect--helper.c
index 1fbe156e67..3055b2bb50 100644
--- a/builtin/bisect--helper.c
+++ b/builtin/bisect--helper.c
@@ -169,11 +169,12 @@ static int bisect_reset(const char *commit)
 
 		argv_array_pushl(&argv, "checkout", branch.buf, "--", NULL);
 		if (run_command_v_opt(argv.argv, RUN_GIT_CMD)) {
+			error(_("could not check out original"
+				" HEAD '%s'. Try 'git bisect"
+				" reset <commit>'."), branch.buf);
 			strbuf_release(&branch);
 			argv_array_clear(&argv);
-			return error(_("could not check out original"
-				       " HEAD '%s'. Try 'git bisect"
-				       " reset <commit>'."), branch.buf);
+			return -1;
 		}
 		argv_array_clear(&argv);
 	}
-- 
2.21.0 (Apple Git-122.2)
Next: Johannes Schindelin
Message 1 of 3 in “bisect--helper: refer branch.buf before strbuf_release(...)”
  1. Miriam RubioDec 8, 2019
  2. Johannes SchindelinDec 8, 2019
  3. Miriam R.Dec 9, 2019

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.