Search Linux Wireless

Re: [PATCH] brcmfmac: fix brcmf_wiphy_wowl_params() NULL pointer dereference

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 08/07/2018 7:28, Kalle Valo wrote:
Chi-Hsien Lin <chi-hsien.lin@xxxxxxxxxxx> writes:

From: Winnie Chang <winnie.chang@xxxxxxxxxxx>

The kernel BUG happens when wowl is enabled from firmware. In
brcmf_wiphy_wowl_params(), cfg is a NULL pointer because it is
drvr->config returned from wiphy_to_cfg(), and drvr->config is not set
yet. To fix it, set drvr->config before brcmf_setup_wiphy() which
calls brcmf_wiphy_wowl_params().

The kernel panic is introduced in below commit:
commit id: 856d5a011c86b59f6564be4508912fb1d866adfc
brcmfmac: allocate struct brcmf_pub instance using wiphy_new()

Signed-off-by: Winnie Chang <winnie.chang@xxxxxxxxxxx>
Reviewed-by: Chi-Hsien Lin <chi-hsien.lin@xxxxxxxxxxx>

Chi-Hsien, you need to include your own s-o-b as well when you are
submitting the patch:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#signed-off-by_missing

Hi Kalle,

Thanks a lot for the notes. I've added proper s-o-b in V2.

Chi-hsien Lin


--
Kalle Valo


----------

You're receiving this message because you're a member of the brcm80211-dev-list group.
.




[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Wireless Regulations]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux