On Sat, 11 Jan 2025 at 20:42, Sam Protsenko <[email protected]> wrote: > > Free memory allocated for 'order' (array of bootmeths) on error paths in > bootmeth_setup_iter_order() function. > > Fixes: c627cfc14c08 ("bootstd: Allow scanning for global bootmeths > separately") > Fixes: 10d16faa436c ("bootstd: Detect empty bootmeth") > Signed-off-by: Sam Protsenko <[email protected]> > --- > boot/bootmeth-uclass.c | 16 ++++++++++++---- > 1 file changed, 12 insertions(+), 4 deletions(-) >
Reviewed-by: Simon Glass <[email protected]> How about adding a test to check there is no leak? You can see test/lib/abuf.c for similar tests.
