On Thu, Aug 26, 2021 at 01:19:21AM +0200, Ævar Arnfjörð Bjarmason wrote: > Fix a memory leak introduced in ee4512ed481 (trace2: create new > combined trace facility, 2019-02-22), we were doing a free() of other > memory allocated in tr2tls_create_self(), but not the "thread_name" > "strbuf strbuf". Your patch is obviously correct, but is "strbuf strbuf" a typo for "struct strbuf"? It doesn't really matter, and it certainly is not worth re-submitting 6 patches for a single typo, but just curious. Thanks, Taylor
