Re: [PATCH v1] hamradio: remove needs_free_netdev to avoid UAF

From: patchwork-bot+netdevbpf
Date: Fri Nov 12 2021 - 23:21:03 EST


Hello:

This patch was applied to netdev/net.git (master)
by Jakub Kicinski <kuba@xxxxxxxxxx>:

On Thu, 11 Nov 2021 22:14:02 +0800 you wrote:
> The former patch "defer 6pack kfree after unregister_netdev" reorders
> the kfree of two buffer after the unregister_netdev to prevent the race
> condition. It also adds free_netdev() function in sixpack_close(), which
> is a direct copy from the similar code in mkiss_close().
>
> However, in sixpack driver, the flag needs_free_netdev is set to true in
> sp_setup(), hence the unregister_netdev() will free the netdev
> automatically. Therefore, as the sp is netdev_priv, use-after-free
> occurs.
>
> [...]

Here is the summary with links:
- [v1] hamradio: remove needs_free_netdev to avoid UAF
https://git.kernel.org/netdev/net/c/81b1d548d00b

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html