On Tue, Mar 29, 2016 at 05:38:51PM -0700, Stefan Beller wrote: > `split` is of type `struct strbuf **` and just before the new free, > we release the inner strbufs. Make sure to also release the memory > containing the pointers to the individual strbufs. > > Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx> > --- > wt-status.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/wt-status.c b/wt-status.c > index ef74864..a78cfc0 100644 > --- a/wt-status.c > +++ b/wt-status.c > @@ -1065,7 +1065,7 @@ static void abbrev_sha1_in_line(struct strbuf *line) > } > for (i = 0; split[i]; i++) > strbuf_release(split[i]); > - > + free(split); > } I think this can just combine with the for-loop above to become strbuf_list_free(). -Peff -- 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