2020-02-03 03:06:37

by Wei Yongjun

[permalink] [raw]
Subject: [PATCH net-next] rtw88: use kfree_skb() instead of kfree()

Use kfree_skb() instead of kfree() to free sk_buff.

Fixes: b6c12908a33e ("rtw88: Add wowlan net-detect support")
Signed-off-by: Wei Yongjun <[email protected]>
---
drivers/net/wireless/realtek/rtw88/fw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/realtek/rtw88/fw.c b/drivers/net/wireless/realtek/rtw88/fw.c
index 243441453ead..5bdfc70c4529 100644
--- a/drivers/net/wireless/realtek/rtw88/fw.c
+++ b/drivers/net/wireless/realtek/rtw88/fw.c
@@ -745,7 +745,7 @@ static struct sk_buff *rtw_nlo_info_get(struct ieee80211_hw *hw)
loc = rtw_get_rsvd_page_probe_req_location(rtwdev, ssid);
if (!loc) {
rtw_err(rtwdev, "failed to get probe req rsvd loc\n");
- kfree(skb);
+ kfree_skb(skb);
return NULL;
}
nlo_hdr->location[i] = loc;




2020-02-12 16:03:55

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH net-next] rtw88: use kfree_skb() instead of kfree()

Wei Yongjun <[email protected]> wrote:

> Use kfree_skb() instead of kfree() to free sk_buff.
>
> Fixes: b6c12908a33e ("rtw88: Add wowlan net-detect support")
> Signed-off-by: Wei Yongjun <[email protected]>

Dan already submitted identical patch:
https://patchwork.kernel.org/patch/11356627/

Patch set to Superseded.

--
https://patchwork.kernel.org/patch/11361999/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches