Skip to content

Commit 3410059

Browse files
CIQ Kernel Automationroxanan1996
authored andcommitted
wifi: brcmfmac: validate bsscfg indices in IF events
jira VULN-185617 cve CVE-2026-43110 commit-author Pengpeng Hou <pengpeng@iscas.ac.cn> commit 304950a upstream-diff | Use `brcmf_err` instead of `bphy_err` due to missing dcb1471 ("brcmfmac: use bphy_err() in all wiphy-related code") brcmf_fweh_handle_if_event() validates the firmware-provided interface index before it touches drvr->iflist[], but it still uses the raw bsscfgidx field as an array index without a matching range check. Reject IF events whose bsscfg index does not fit in drvr->iflist[] before indexing the interface array. Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn> Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com> Link: https://patch.msgid.link/20260323074551.93530-1-pengpeng@iscas.ac.cn [add missing wifi prefix] Signed-off-by: Johannes Berg <johannes.berg@intel.com> (cherry picked from commit 304950a) Signed-off-by: CIQ Kernel Automation <ciq_kernel_automation@ciq.com> Signed-off-by: Roxana Nicolescu <rnicolescu@ciq.com>
1 parent 5dad14f commit 3410059

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

  • drivers/net/wireless/broadcom/brcm80211/brcmfmac

drivers/net/wireless/broadcom/brcm80211/brcmfmac/fweh.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -161,6 +161,10 @@ static void brcmf_fweh_handle_if_event(struct brcmf_pub *drvr,
161161
brcmf_err("invalid interface index: %u\n", ifevent->ifidx);
162162
return;
163163
}
164+
if (ifevent->bsscfgidx >= BRCMF_MAX_IFS) {
165+
brcmf_err("invalid bsscfg index: %u\n", ifevent->bsscfgidx);
166+
return;
167+
}
164168

165169
ifp = drvr->iflist[ifevent->bsscfgidx];
166170

0 commit comments

Comments
 (0)