From 3945264468ebd374d5db98758f9428822ef78251 Mon Sep 17 00:00:00 2001 From: Lorenzo Bianconi Date: Thu, 27 May 2021 13:35:28 +0200 Subject: [PATCH] mt76: testmode: fix memory leak in mt76_testmode_alloc_skb Free all pending frames in case of failure in mt76_testmode_alloc_skb routine Fixes: 2601dda8faa76 ("mt76: testmode: add support to send larger packet") Signed-off-by: Lorenzo Bianconi Signed-off-by: Felix Fietkau --- testmode.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/testmode.c b/testmode.c index 001d0ba5..f40387a8 100644 --- a/testmode.c +++ b/testmode.c @@ -158,8 +158,11 @@ int mt76_testmode_alloc_skb(struct mt76_phy *phy, u32 len) frag_len = MT_TXP_MAX_LEN; frag = alloc_skb(frag_len, GFP_KERNEL); - if (!frag) + if (!frag) { + mt76_testmode_free_skb(phy); + dev_kfree_skb(head); return -ENOMEM; + } __skb_put_zero(frag, frag_len); head->len += frag->len;