Martin Ågren <martin.agren@xxxxxxxxx> writes: > On Sun, 25 Sept 2022 at 00:53, Rubén Justo <rjusto@xxxxxxxxx> wrote: >> >> In 2708ce62d2 (branch: sort detached HEAD based on a flag, 2021-01-07) a >> call to wt_status_state_free_buffers, responsible of freeing the >> resources that could be allocated in the local struct wt_status_state >> state, was eliminated. >> >> The call to wt_status_state_free_buffers was introduced in 962dd7ebc3 >> (wt-status: introduce wt_status_state_free_buffers(), 2020-09-27). This >> commit brings back that call in get_head_description. > >> + wt_status_state_free_buffers(&state); >> + >> return strbuf_detach(&desc, NULL); >> } > > Good catch, and excellent history digging. From the original submission > [1] of the patch that dropped this call, I get the feeling that it was > originally developed some time earlier. I suspect this call was then > accidentally dropped in a rebase before submission. > > FWIW, this patch is > > Reviewed-by: Martin Ågren <martin.agren@xxxxxxxxx> > > [1] https://lore.kernel.org/git/20210106100139.14651-1-avarab@xxxxxxxxx/ > > Martin Thanks, all. Will queue lest I forget, but I presume this is not release critical?