Miklos Vajna <vmiklos@xxxxxxxxxxxxxx> writes: > Signed-off-by: Miklos Vajna <vmiklos@xxxxxxxxxxxxxx> > --- > builtin-clone.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/builtin-clone.c b/builtin-clone.c > index 8e1a1d3..da21cab 100644 > --- a/builtin-clone.c > +++ b/builtin-clone.c > @@ -516,6 +516,7 @@ int cmd_clone(int argc, const char **argv, const char *prefix) > refs = transport_get_remote_refs(transport); > transport_fetch_refs(transport, refs); > } > + free(dir); > > clear_extra_refs(); Can't this be done much earlier? This variable can potentially be assigned to work_tree, but after we set up the atexit handler neither dir nor work_tree are not used (it is a bit hard to see as this function itself is a bit too big to be maintainable). -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
