On Sun, 2019-12-08 at 07:07 -0600, Hari Nagalla wrote:
>
> +static inline bool ieee80211_is_not_group_privacy(struct ieee80211_hdr *hdr)
> +{
> + if (ieee80211_is_action(hdr->frame_control)) {
> + u8 *category;
> +
> + /*
> + * Action frames, excluding Public Action frames, are Robust
> + * Management Frames. However, if we are looking at a Protected
> + * frame, skip the check since the data may be encrypted and
> + * the frame has already been found to be a Robust Management
> + * Frame (by the other end).
> + */
I think you copy/pasted that comment a bit too blindly :)
Maybe inverting the function ("is" vs. "is not") would make sense?
johannes
