RE: [PATCH] HE: fix he_capabilities size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I agree that https://patchwork.ozlabs.org/patch/1116968/ can replace both:
https://patchwork.ozlabs.org/patch/1109462/
https://patchwork.ozlabs.org/patch/1122057/

Thanks,

Shay Bar
SW Engineer
Celeno Communications (tm)

shay.bar@xxxxxxxxxx

The information transmitted is intended only for the person or entity to which it is addressed and may contain confidential and/or privileged material. Any retransmission, dissemination, copying or other use of, or taking of any action in reliance upon this information is prohibited.  If you received this in error, please contact the sender and delete the material from any computer. Nothing contained herein shall be deemed as a representation, warranty or a commitment by Celeno. No warranties are expressed or implied, including, but not limited to, any implied warranties of non-infringement, merchantability and fitness for a particular purpose.



-----Original Message-----
From: Sven Eckelmann <sven@xxxxxxxxxxxxx>
Sent: Thursday, 27 June 2019 11:12
To: hostap@xxxxxxxxxxxxxxxxxxx
Cc: John Crispin <john@xxxxxxxxxxx>; Shay Bar <Shay.Bar@xxxxxxxxxx>
Subject: Re: [PATCH] HE: fix he_capabilities size

On Monday, 3 June 2019 21:21:17 CEST John Crispin wrote:
> The ppet field inside ieee80211_he_capabilities is of size [0]. The
> code currently copies up to 12 additional bytes into the buffer, thus
> overwriting memory. Fix this by verifying the size properly and using
> the passed length value for allocation and the following memcpy() call.
>
> Signed-off-by: John Crispin <john@xxxxxxxxxxx>

I just went to the patches which might be interesting for me to get HE in a working state for ath11k mesh. And so I found following patches which seemed to be relevant for me:

* https://patchwork.ozlabs.org/patch/1109462/ (this patched; marked as
  "changes requested")
* https://patchwork.ozlabs.org/patch/1122057/ (marked superseded)

It looks to me that the mentioned patches can all be replaced by https://patchwork.ozlabs.org/patch/1116968/ - is this assumption correct or am I missing something?

[...]
> diff --git a/src/ap/ieee802_11_he.c b/src/ap/ieee802_11_he.c index
> ba22a174a..63270228f 100644
> --- a/src/ap/ieee802_11_he.c
> +++ b/src/ap/ieee802_11_he.c
> @@ -44,6 +44,39 @@ static u8 ieee80211_he_ppet_size(u8 ppe_thres_hdr,
> const
u8 *phy_cap_info)
>  }
>
>
> +static inline u8
> +ieee80211_he_mcs_set_size(const u8 *phy_cap_info) {

We talked about this on a call (were I had problems to understand you) and I was under the impression that you were talking about static inline functions in headers. So I have to retract my statement and agree with Jouni about this.

Kind regards,
Sven
________________________________
The information transmitted is intended only for the person or entity to which it is addressed and may contain confidential and/or privileged material. Any retransmission, dissemination, copying or other use of, or taking of any action in reliance upon this information is prohibited. If you received this in error, please contact the sender and delete the material from any computer. Nothing contained herein shall be deemed as a representation, warranty or a commitment by Celeno. No warranties are expressed or implied, including, but not limited to, any implied warranties of non-infringement, merchantability and fitness for a particular purpose.
________________________________


_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux