]> git.baikalelectronics.ru Git - kernel.git/commit
mac80211: fix deadlock in AP/VLAN handling
authorJohannes Berg <johannes.berg@intel.com>
Mon, 17 May 2021 14:03:23 +0000 (16:03 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Tue, 8 Jun 2021 09:33:07 +0000 (11:33 +0200)
commite5931775818afe3d2c2bd0511318ed0337b0c9f6
treed02b071713fea8c3df26ce8ac33dd05c9695b9d1
parentae97ff3e7bcda7f0bc7318c91c0277b6819f2c7e
mac80211: fix deadlock in AP/VLAN handling

Syzbot reports that when you have AP_VLAN interfaces that are up
and close the AP interface they belong to, we get a deadlock. No
surprise - since we dev_close() them with the wiphy mutex held,
which goes back into the netdev notifier in cfg80211 and tries to
acquire the wiphy mutex there.

To fix this, we need to do two things:
 1) prevent changing iftype while AP_VLANs are up, we can't
    easily fix this case since cfg80211 already calls us with
    the wiphy mutex held, but change_interface() is relatively
    rare in drivers anyway, so changing iftype isn't used much
    (and userspace has to fall back to down/change/up anyway)
 2) pull the dev_close() loop over VLANs out of the wiphy mutex
    section in the normal stop case

Cc: stable@vger.kernel.org
Reported-by: syzbot+452ea4fbbef700ff0a56@syzkaller.appspotmail.com
Fixes: 03d71e9f6564 ("cfg80211: avoid holding the RTNL when calling the driver")
Link: https://lore.kernel.org/r/20210517160322.9b8f356c0222.I392cb0e2fa5a1a94cf2e637555d702c7e512c1ff@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/iface.c