On Fri, 2021-08-27 at 10:11 +0800, Wen Gong wrote: > > Also, if we're doing this anyway, then we can change > > the code above (perhaps as a separate patch) to not do > > cfg80211_find_ext_ie() but rather take it out of the parsed. > do you mean NOT use cfg80211_find_ext_ie()/cfg80211_find_ie() and still > use "struct ieee802_11_elems elems" here and > move this code to a separate function/patch? Well, there's an existing place in this function that uses cfg80211_find_ext_ie(), and various uses of ieee80211_bss_get_ie(), so it feels like if we're going to do the full parsing, we should switch all the existing "look up an element" to also use the parsed data instead. Not the other way around :) johannes
