Re: [PATCH] mt7601u: add missing release on skb in mt7601u_mcu_msg_send

2020-08-02 Thread Kalle Valo
Navid Emamdoost wrote: > In the implementation of mt7601u_mcu_msg_send(), skb is supposed to be > consumed on all execution paths. Release skb before returning if > test_bit() fails. > > Signed-off-by: Navid Emamdoost > Acked-by: Jakub Kicinski Patch applied to wireless-drivers-next.git, than

Re: [PATCH] mt7601u: add missing release on skb in mt7601u_mcu_msg_send

2020-07-20 Thread Jakub Kicinski
On Sat, 18 Jul 2020 00:26:29 -0500 Navid Emamdoost wrote: > In the implementation of mt7601u_mcu_msg_send(), skb is supposed to be > consumed on all execution paths. Release skb before returning if > test_bit() fails. > > Signed-off-by: Navid Emamdoost Acked-by: Jakub Kicinski

Re: [PATCH] mt7601u: add missing release on skb in mt7601u_mcu_msg_send

2020-07-18 Thread Markus Elfring
… > +++ b/drivers/net/wireless/mediatek/mt7601u/mcu.c > @@ -116,8 +116,10 @@ mt7601u_mcu_msg_send(struct mt7601u_dev *dev, struct > sk_buff *skb, > int sent, ret; > u8 seq = 0; > > - if (test_bit(MT7601U_STATE_REMOVED, &dev->state)) > + if (test_bit(MT7601U_STATE_REMOVED, &dev-

[PATCH] mt7601u: add missing release on skb in mt7601u_mcu_msg_send

2020-07-17 Thread Navid Emamdoost
In the implementation of mt7601u_mcu_msg_send(), skb is supposed to be consumed on all execution paths. Release skb before returning if test_bit() fails. Signed-off-by: Navid Emamdoost --- drivers/net/wireless/mediatek/mt7601u/mcu.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff