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