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
