On 11/21, Junio C Hamano wrote:
It is rather common for us to reuse "struct child_process" in a code
path, e.g. builtin/worktree.c::add_worktree() prepares a single
instance of such a struct, sets cp.git_cmd to true, and runs either
"update-ref" or "symbolic-ref" to update "HEAD". After successful
invocation of such a git subcommand, it then runs "git reset --hard",
with this piece of code:
Do you agree that at least NULLing .argv and .env could be part of
child_process_clear()?
diff --git a/run-command.c b/run-command.c
index a7bf81025afb..3839a26eff11 100644
--- a/run-command.c
+++ b/run-command.c
@@ -18,8 +18,9 @@ void child_process_init(struct child_process *child)
void child_process_clear(struct child_process *child)
{
strvec_clear(&child->args);
+ child->argv = NULL;
strvec_clear(&child->env_array);
+ child->env = NULL;
}
With this change on main, all tests are successful.
Cheers,
Enzo
