On Tue, Mar 10, 2020 at 06:20:43PM +0000, brian m. carlson wrote: > diff --git a/builtin/clone.c b/builtin/clone.c > index 1ad26f4d8c..00e5427ef1 100644 > --- a/builtin/clone.c > +++ b/builtin/clone.c > @@ -780,11 +780,12 @@ static int checkout(int submodule_progress) > if (!strcmp(head, "HEAD")) { > if (advice_detached_head) > detach_advice(oid_to_hex(&oid)); > + free(head); > + head = NULL; Coccinelle suggests to use FREE_AND_NULL(head) instead.