sub-process: avoid leaking cmd

In some instances (particularly the `read_object` hook), the `cmd`
attribute is set to an `strdup()`ed value. This value needs to be
released in the end!

Since other users assign a non-`strdup()`ed value, be careful to add
_another_ attribute (called `to_free`) that can hold a reference to such
a string that needs to be released once the sub process is done.

Signed-off-by: Johannes Schindelin <[email protected]>

Author: dscho

sub-process.c

@@ -63,6 +63,8 @@ void subprocess_stop(struct hashmap *hashmap, struct subprocess_entry *entry)
 	finish_command(&entry->process);
 
 	hashmap_remove(hashmap, &entry->ent, NULL);
+	FREE_AND_NULL(entry->to_free);
+	entry->cmd = NULL;
 }
 
 static void subprocess_exit_handler(struct child_process *process)
@@ -145,11 +147,13 @@ int subprocess_start_strvec(struct hashmap *hashmap,
 	process->trace2_child_class = "subprocess";
 
 	sq_quote_argv_pretty(&quoted, argv->v);
-	entry->cmd = strbuf_detach(&quoted, NULL);
+	entry->cmd = entry->to_free = strbuf_detach(&quoted, NULL);
 
 	err = start_command(process);
 	if (err) {
 		error("cannot fork to run subprocess '%s'", entry->cmd);
+		FREE_AND_NULL(entry->to_free);
+		entry->cmd = NULL;
 		return err;
 	}
 
@@ -158,6 +162,8 @@ int subprocess_start_strvec(struct hashmap *hashmap,
 	err = startfn(entry);
 	if (err) {
 		error("initialization for subprocess '%s' failed", entry->cmd);
+		FREE_AND_NULL(entry->to_free);
+		entry->cmd = NULL;
 		subprocess_stop(hashmap, entry);
 		return err;
 	}

sub-process.h

@@ -25,6 +25,12 @@
 struct subprocess_entry {
 	struct hashmap_entry ent;
 	const char *cmd;
+	/**
+	 * In case `cmd` is a `strdup()`ed value that needs to be released,
+	 * you can assign the pointer to `to_free` so that `subprocess_stop()`
+	 * will release it.
+	 */
+	char *to_free;
 	struct child_process process;
 };