[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <fd8da5d5-8e11-4e88-bef3-3e50dad72aa6@bootlin.com>
Date: Wed, 12 Feb 2025 10:05:17 +0100
From: Bastien Curutchet <bastien.curutchet@...tlin.com>
To: Kuniyuki Iwashima <kuniyu@...zon.com>
Cc: alexis.lothore@...tlin.com, davem@...emloft.net, edumazet@...gle.com,
horms@...nel.org, kuba@...nel.org, linux-kernel@...r.kernel.org,
netdev@...r.kernel.org, pabeni@...hat.com, razor@...ckwall.org,
stable@...r.kernel.org, thomas.petazzoni@...tlin.com
Subject: Re: [PATCH net 2/2] rtnetlink: Release nets when leaving
rtnl_setlink()
On 2/12/25 9:31 AM, Kuniyuki Iwashima wrote:
> From: "Bastien Curutchet (eBPF Foundation)" <bastien.curutchet@...tlin.com>
> Date: Wed, 12 Feb 2025 09:23:48 +0100
>> rtnl_setlink() uses the rtnl_nets_* helpers but never calls the
>> rtnl_nets_destroy(). It leads to small memory leaks.
>>
>> Call rtnl_nets_destroy() before exiting to properly decrement the nets'
>> reference counters.
>>
>> Fixes: 636af13f213b ("rtnetlink: Register rtnl_dellink() and rtnl_setlink() with RTNL_FLAG_DOIT_PERNET_WIP.")
>> Cc: stable@...r.kernel.org
>> Signed-off-by: Bastien Curutchet (eBPF Foundation) <bastien.curutchet@...tlin.com>
>
> It's fixed in 1438f5d07b9a ("rtnetlink: fix netns leak with
> rtnl_setlink()").
>
Oops, I missed it, sorry about that.
Best regards,
Bastien
Powered by blists - more mailing lists