diff options
| author | Arend Van Spriel <arend.vanspriel@broadcom.com> | 2017-11-25 21:39:25 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-12-14 09:28:13 +0100 |
| commit | 4a70f07da85313aa817c0888d2b623ea30cbbbe5 (patch) | |
| tree | 42966a1ea5904723886e986e6a6045d2df3d68f2 /kernel/debug/kdb | |
| parent | 8950c982fffd9700ca7ef2fc2586638e47669f19 (diff) | |
| download | linux-4a70f07da85313aa817c0888d2b623ea30cbbbe5.tar.gz linux-4a70f07da85313aa817c0888d2b623ea30cbbbe5.tar.bz2 linux-4a70f07da85313aa817c0888d2b623ea30cbbbe5.zip | |
brcmfmac: change driver unbind order of the sdio function devices
commit 5c3de777bdaf48bd0cfb43097c0d0fb85056cab7 upstream.
In the function brcmf_sdio_firmware_callback() the driver is
unbound from the sdio function devices in the error path.
However, the order in which it is done resulted in a use-after-free
issue (see brcmf_ops_sdio_remove() in bcmsdh.c). Hence change
the order and first unbind sdio function #2 device and then
unbind sdio function #1 device.
Fixes: 7a51461fc2da ("brcmfmac: unbind all devices upon failure in firmware callback")
Reported-by: Stefan Wahren <stefan.wahren@i2se.com>
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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'kernel/debug/kdb')
0 files changed, 0 insertions, 0 deletions
