Re: [PATCH v6] wifi: brcmfmac: cyw: pass PMKID to firmware if present
Bogdan Nicolae <[email protected]>
| Newsgroups | dev.linux.lists.brcm80211,org.kernel.vger.linux-kernel,org.kernel.vger.linux-wireless,org.kernel.vger.stable |
|---|---|
| Message-ID | <CA+ORkNRRZ_qES77+12=Cx2rs_ntVqmOB0vzUtTOJddabpMvAyg@mail.gmail.com> |
Sure! Best, Bogdan On Fri, Aug 7, 2026 at 11:52 AM Arend van Spriel <[email protected]> wrote: > > Op 7 augustus 2026 18:37:48 schreef Bogdan Nicolae <[email protected]>: > > > Zero out auth_status on initialization. Otherwise, garbage will > > leak from the stack to the firmware (when ssid is less than 32 bytes > > and/or when params->pmkid is set). Then, pass the params->pmkid to the > > firmware (without it, the firmware caches a garbage PMKID on successful > > authentication and denies a subsequent association request that includes > > the PMKID). > > > > Fixes: 66f909308a7c ("wifi: brcmfmac: cyw: support external SAE > > authentication in station mode") > > Signed-off-by: Bogdan Nicolae <[email protected]> > > Acked-by: Arend van Spriel <[email protected]> > > Hi Bogdan, > > Looks good so my Acked-by: still stands. Thanks for going with the flow. > > Regards, > Arend > > > --- > > v6: Fixed diff line number > > v5: Fixed line length > 80 chars > > v4: Fixed indentation > > v3: Added Fixes, Cc: stable, and Acked-by tags > > v2: Split into a separate patch > > > > drivers/net/wireless/broadcom/brcm80211/brcmfmac/cyw/core.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > >