On Monday, 17 June 2019 14:16:35 CEST John Crispin wrote:
[...]
> @@ -63,7 +63,7 @@ ieee80211_he_mcs_set_size(const u8 *phy_cap_info)
> static inline int ieee80211_check_he_cap_size(const u8 *buf, int len)
> {
> struct ieee80211_he_capabilities *cap = (struct ieee80211_he_capabilities *)buf;
> - int cap_len = sizeof(struct ieee80211_he_capabilities);
> + int cap_len = sizeof(*cap) - sizeof(cap->optional);
>
> if (len < cap_len)
> return 1;
Since it was discussed that this patch replaces "HE: fix he_capabilities
size" [1], it should maybe not modify a function which was first introduced in
a patch which is now getting replaced.
Kind regards,
Sven
[1] https://patchwork.ozlabs.org/patch/1109462/Attachment:
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap
