On Mon, 2024-05-06 at 14:20 -0700, Muna Sinada wrote:
>
> + if (sta && sdata->vif.type == NL80211_IFTYPE_AP_VLAN) {
> + ap_sdata = container_of(sdata->bss,
> + struct ieee80211_sub_if_data, u.ap);
> + if (ap_sdata->vif.offload_flags & IEEE80211_OFFLOAD_ENCAP_ENABLED &&
> + !is_multicast_ether_addr(skb->data)) {
> + key = rcu_dereference(sta->ptk[sta->ptk_idx]);
Why is that line there, and why is it necessary? Is it even correct?
I see you need a key pointer on the next line, but still... that doesn't
seem right.
johannes
