"Phillip Wood via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes: > ... This > introduces a memory leak similar to the one leaking GIT_REFLOG_ACTION > in pick_commits(). Is it just the matter of freeing previous_reflog_action after you call setenv(), or does it take much more involved changes? > Both of these will be fixed in a future series that > stops the sequencer calling setenv(). If it gets fixed in a future step in the same series, that is a different matter, but if it is easy enough not to deliberately introduce a new leak, we'd prefer to do so.