brcmfmac: fix sdio sg table alloc crash
authorHante Meuleman <meuleman@broadcom.com>
Tue, 19 Jan 2016 11:39:24 +0000 (12:39 +0100)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 19 Jan 2016 19:27:22 +0000 (21:27 +0200)
commite0045bf80f62779ba75ba04513020d60e59f5298
tree72f7bf06e3501073a34c4aa1cd0f0d19c38dbb2d
parentb7e2d195cc523bd92ed7a64aac089421dbab2a2d
brcmfmac: fix sdio sg table alloc crash

With commit 7d34b0560567 ("brcmfmac: Move all module parameters to
one place") a bug was introduced causing a null pointer exception.
This patch fixes the bug by initializing the sg table till after
the settings have been initialized.

Fixes: 7d34b0560567 ("brcmfmac: Move all module parameters to one place")
Reported-by: Marc Zyngier <marc.zyngier@arm.com>
Tested-by: Marc Zyngier <marc.zyngier@arm.com>
Reviewed-by: Arend Van Spriel <arend@broadcom.com>
Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Hante Meuleman <meuleman@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c
drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c
drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.h