On 11/25/21 2:04 PM, Mateusz Grzonka wrote: >On 11/24/21 5:30 PM, Jes Sorensen wrote: >>On 11/24/21 8:27 AM, Tkaczyk, Mariusz wrote: >>> On 24.11.2021 13:13, Jes Sorensen wrote: >>>> On 11/24/21 6:58 AM, Mateusz Grzonka wrote: >>>>> map allocated through map_by_uuid() is not freed if mdfd is invalid. >>>>> In addition mdfd is not closed, and mdinfo list is not freed too. >>>>> >>>>> Signed-off-by: Mateusz Grzonka <mateusz.grzonka@xxxxxxxxx> >>>>> --- >>>>> Incremental.c | 32 +++++++++++++++++++++++--------- >>>>> 1 file changed, 23 insertions(+), 9 deletions(-) >>>> >>>> I already applied the previous version. Could you please send an >>>> updated version on top of current tree. >>>> >>>> Thanks, >>>> Jes >>>> >>> >>> Hi Jes, >>> I cannot see previous version in mdadm tree. >>> Could you verify? >> >> You may have been too quick and it hadn't propagated yet. Mind trying one more time? >> >> Thanks, >> Jes > > Hi Jes, > I split previous "[PATCH] Incremental: Fix possible memory and resource leaks" into 3 separate patches as suggested. > Two of them, that were created as a result of cutting things from "[PATCH] Incremental: Fix possible memory and resource leaks", were merged yesterday. > However this patch (as well as its previous version) was not. > > Regards, > Mateusz Hi Jes, any updates on this one?
