On Mon, Jun 21, 2021 at 11:54 PM Elijah Newren <newren@xxxxxxxxx> wrote: > > On Sun, Jun 20, 2021 at 8:14 AM <andrzej@xxxxxxxxx> wrote: > > > > From: Andrzej Hunt <andrzej@xxxxxxxxx> > > > > This series plugs more of the leaks that were found while running > > t0002-t0099 with LSAN. > > > > See also the first series (already merged) at [1]. I'm currently > > expecting at least another 2 series before t0002-t0099 run leak free. > > I'm not being particularly systematic about the order of patches - > > although I am trying to send out "real" (if mostly small) leaks first, > > before sending out the more boring patches that add free()/UNLEAK() to > > cmd_* and direct helpers thereof. > > I've read over the series. It provides some good clear fixes. I > noted on patches 2, 6, and 12 that a some greps suggested that leaks > similar to the ones being fixed likely also affect other places of the > codebase. Those other places don't need to be fixed as part of this > series, but they might be good items for #leftoverbits or GSoC early > tasks (cc: Christian in case he wants to record those somewhere). Yeah, thanks for letting me know!