Ziyang Xuan f7fc72a508 net: vlan: fix underflow for the real_dev refcnt ... [ Upstream commit 01d9cc2dea ] Inject error before dev_hold(real_dev) in register_vlan_dev(), and execute the following testcase: ip link add dev dummy1 type dummy ip link add name dummy1.100 link dummy1 type vlan id 100 ip link del dev dummy1 When the dummy netdevice is removed, we will get a WARNING as following: ======================================================================= refcount_t: decrement hit 0; leaking memory. WARNING: CPU: 2 PID: 0 at lib/refcount.c:31 refcount_warn_saturate+0xbf/0x1e0 and an endless loop of: ======================================================================= unregister_netdevice: waiting for dummy1 to become free. Usage count = -1073741824 That is because dev_put(real_dev) in vlan_dev_free() be called without dev_hold(real_dev) in register_vlan_dev(). It makes the refcnt of real_dev underflow. Move the dev_hold(real_dev) to vlan_dev_init() which is the call-back of ndo_init(). That makes dev_hold() and dev_put() for vlan's real_dev symmetrical. Fixes: 563bcbae3b ("net: vlan: fix a UAF in vlan_dev_real_dev()") Reported-by: Petr Machata <petrm@nvidia.com> Suggested-by: Jakub Kicinski <kuba@kernel.org> Signed-off-by: Ziyang Xuan <william.xuanziyang@huawei.com> Link: https://lore.kernel.org/r/20211126015942.2918542-1-william.xuanziyang@huawei.com Signed-off-by: Jakub Kicinski <kuba@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>		2021-12-01 09:04:53 +01:00
..
Kconfig	treewide: replace '---help---' in Kconfig files with 'help'	2020-06-14 01:57:21 +09:00
Makefile
vlan.c	net: vlan: fix underflow for the real_dev refcnt	2021-12-01 09:04:53 +01:00
vlan.h	net: vlan: pass thru all GSO_SOFTWARE in hw_enc_features	2021-06-18 11:58:03 -07:00
vlan_core.c	vlan/8021q: avoid retpoline overhead on GRO	2021-03-18 19:51:12 -07:00
vlan_dev.c	net: vlan: fix underflow for the real_dev refcnt	2021-12-01 09:04:53 +01:00
vlan_gvrp.c
vlan_mvrp.c
vlan_netlink.c
vlanproc.c
vlanproc.h