Messages in this thread | | | Subject | Re: [PATCH] net: mac80211: cfg: enforce sanity checks for key_index in ieee80211_del_key() | From | Johannes Berg <> | Date | Tue, 01 Dec 2020 13:56:13 +0100 |
| |
On Tue, 2020-12-01 at 18:15 +0530, Anant Thazhemadam wrote: > > cfg80211_supported_cipher_suite(&rdev->wiphy, params->cipher) returned > false, and thus it worked for the syzbot reproducer. > Would it be a safer idea to enforce the conditions that I initially put (in > ieee80211_del_key()) directly in cfg80211_validate_key_settings() itself - by > updating max_key_index, and checking accordingly?
Yes, I think so. But similarly to cfg80211_validate_key_settings() it should look at the device capabilities (beacon protection, etc.)
Thanks! johannes
| |