brcmfmac: add length check in brcmf_cfg80211_escan_handler()

Browse source

commit 17df6453d4 upstream.

Upon handling the firmware notification for scans the length was
checked properly and may result in corrupting kernel heap memory
due to buffer overruns. This fix addresses CVE-2017-0786.

Cc: Kevin Cernekee <cernekee@chromium.org>
Reviewed-by: Hante Meuleman <hante.meuleman@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieter-paul.giesberts@broadcom.com>
Reviewed-by: Franky Lin <franky.lin@broadcom.com>
Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
[bwh: Backported to 3.16:
 - Use WL_ESCAN_BUF_SIZE instead of BRCMF_ESCAN_BUF_SIZE
 - Adjust filename]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

...

This commit is contained in:

Arend Van Spriel 2017-09-12 10:47:53 +02:00 • committed by Ben Hutchings

parent 8a97fcd83a

commit 7df83adfc5

1 changed files with 15 additions and 3 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

18

drivers/net/wireless/brcm80211/brcmfmac/wl_cfg80211.c

View file