brcmfmac: Fix double freeing in the fmac usb data path
authorRaveendran Somu <raveendran.somu@cypress.com>
Wed, 25 Mar 2020 08:20:25 +0000 (03:20 -0500)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 26 Mar 2020 09:43:53 +0000 (11:43 +0200)
commit78179869dc3f5c0059bbf5d931a2717f1ad97ecd
treed281e4e3f0819ac285eb4b718ad1311e648ba3ec
parent93a5bfbc7cad8bf3dea81c9bc07761c1226a0860
brcmfmac: Fix double freeing in the fmac usb data path

When the brcmf_fws_process_skb() fails to get hanger slot for
queuing the skb, it tries to free the skb.
But the caller brcmf_netdev_start_xmit() of that funciton frees
the packet on error return value.
This causes the double freeing and which caused the kernel crash.

Signed-off-by: Raveendran Somu <raveendran.somu@cypress.com>
Signed-off-by: Chi-hsien Lin <chi-hsien.lin@cypress.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/1585124429-97371-3-git-send-email-chi-hsien.lin@cypress.com
drivers/net/wireless/broadcom/brcm80211/brcmfmac/fwsignal.c