[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id:
<173249583150.3408383.17756861808285881357.git-patchwork-notify@kernel.org>
Date: Mon, 25 Nov 2024 00:50:31 +0000
From: patchwork-bot+netdevbpf@...nel.org
To: Eric Dumazet <edumazet@...gle.com>
Cc: davem@...emloft.net, kuba@...nel.org, pabeni@...hat.com,
netdev@...r.kernel.org, eric.dumazet@...il.com, syzkaller@...glegroups.com,
kuniyu@...zon.com
Subject: Re: [PATCH net] rtnetlink: fix rtnl_dump_ifinfo() error path
Hello:
This patch was applied to netdev/net.git (main)
by Jakub Kicinski <kuba@...nel.org>:
On Thu, 21 Nov 2024 19:41:05 +0000 you wrote:
> syzbot found that rtnl_dump_ifinfo() could return with a lock held [1]
>
> Move code around so that rtnl_link_ops_put() and put_net()
> can be called at the end of this function.
>
> [1]
> WARNING: lock held when returning to user space!
> 6.12.0-rc7-syzkaller-01681-g38f83a57aa8e #0 Not tainted
> syz-executor399/5841 is leaving the kernel with locks still held!
> 1 lock held by syz-executor399/5841:
> #0: ffffffff8f46c2a0 (&ops->srcu#2){.+.+}-{0:0}, at: rcu_lock_acquire include/linux/rcupdate.h:337 [inline]
> #0: ffffffff8f46c2a0 (&ops->srcu#2){.+.+}-{0:0}, at: rcu_read_lock include/linux/rcupdate.h:849 [inline]
> #0: ffffffff8f46c2a0 (&ops->srcu#2){.+.+}-{0:0}, at: rtnl_link_ops_get+0x22/0x250 net/core/rtnetlink.c:555
>
> [...]
Here is the summary with links:
- [net] rtnetlink: fix rtnl_dump_ifinfo() error path
https://git.kernel.org/netdev/net/c/9b234a97b10c
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
Powered by blists - more mailing lists