|
Takashi Iwai |
c3ee3e |
From 2283f79b22684d2812e5c76fc2280aae00390365 Mon Sep 17 00:00:00 2001
|
|
Takashi Iwai |
c3ee3e |
From: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
|
|
Takashi Iwai |
c3ee3e |
Date: Sat, 3 Oct 2020 00:41:45 +0900
|
|
Takashi Iwai |
c3ee3e |
Subject: [PATCH] can: dev: can_get_echo_skb(): prevent call to kfree_skb() in hard IRQ context
|
|
Takashi Iwai |
c3ee3e |
Git-commit: 2283f79b22684d2812e5c76fc2280aae00390365
|
|
Takashi Iwai |
c3ee3e |
Patch-mainline: v5.10-rc3
|
|
Takashi Iwai |
c3ee3e |
References: git-fixes
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
If a driver calls can_get_echo_skb() during a hardware IRQ (which is often, but
|
|
Takashi Iwai |
c3ee3e |
not always, the case), the 'WARN_ON(in_irq)' in
|
|
Takashi Iwai |
c3ee3e |
net/core/skbuff.c#skb_release_head_state() might be triggered, under network
|
|
Takashi Iwai |
c3ee3e |
congestion circumstances, together with the potential risk of a NULL pointer
|
|
Takashi Iwai |
c3ee3e |
dereference.
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
The root cause of this issue is the call to kfree_skb() instead of
|
|
Takashi Iwai |
c3ee3e |
dev_kfree_skb_irq() in net/core/dev.c#enqueue_to_backlog().
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
This patch prevents the skb to be freed within the call to netif_rx() by
|
|
Takashi Iwai |
c3ee3e |
incrementing its reference count with skb_get(). The skb is finally freed by
|
|
Takashi Iwai |
c3ee3e |
one of the in-irq-context safe functions: dev_consume_skb_any() or
|
|
Takashi Iwai |
c3ee3e |
dev_kfree_skb_any(). The "any" version is used because some drivers might call
|
|
Takashi Iwai |
c3ee3e |
can_get_echo_skb() in a normal context.
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
The reason for this issue to occur is that initially, in the core network
|
|
Takashi Iwai |
c3ee3e |
stack, loopback skb were not supposed to be received in hardware IRQ context.
|
|
Takashi Iwai |
c3ee3e |
The CAN stack is an exeption.
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
This bug was previously reported back in 2017 in [1] but the proposed patch
|
|
Takashi Iwai |
c3ee3e |
never got accepted.
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
While [1] directly modifies net/core/dev.c, we try to propose here a
|
|
Takashi Iwai |
c3ee3e |
smoother modification local to CAN network stack (the assumption
|
|
Takashi Iwai |
c3ee3e |
behind is that only CAN devices are affected by this issue).
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
[1] http://lore.kernel.org/r/57a3ffb6-3309-3ad5-5a34-e93c3fe3614d@cetitec.com
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
|
|
Takashi Iwai |
c3ee3e |
Link: https://lore.kernel.org/r/20201002154219.4887-2-mailhol.vincent@wanadoo.fr
|
|
Takashi Iwai |
c3ee3e |
Fixes: 39549eef3587 ("can: CAN Network device driver and Netlink interface")
|
|
Takashi Iwai |
c3ee3e |
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
|
|
Takashi Iwai |
c3ee3e |
Acked-by: Takashi Iwai <tiwai@suse.de>
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
---
|
|
Takashi Iwai |
c3ee3e |
drivers/net/can/dev.c | 6 +++++-
|
|
Takashi Iwai |
c3ee3e |
1 file changed, 5 insertions(+), 1 deletion(-)
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
diff --git a/drivers/net/can/dev.c b/drivers/net/can/dev.c
|
|
Takashi Iwai |
c3ee3e |
index b70ded3760f2..73cfcd7e9517 100644
|
|
Takashi Iwai |
c3ee3e |
--- a/drivers/net/can/dev.c
|
|
Takashi Iwai |
c3ee3e |
+++ b/drivers/net/can/dev.c
|
|
Takashi Iwai |
c3ee3e |
@@ -538,7 +538,11 @@ unsigned int can_get_echo_skb(struct net_device *dev, unsigned int idx)
|
|
Takashi Iwai |
c3ee3e |
if (!skb)
|
|
Takashi Iwai |
c3ee3e |
return 0;
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
- netif_rx(skb);
|
|
Takashi Iwai |
c3ee3e |
+ skb_get(skb);
|
|
Takashi Iwai |
c3ee3e |
+ if (netif_rx(skb) == NET_RX_SUCCESS)
|
|
Takashi Iwai |
c3ee3e |
+ dev_consume_skb_any(skb);
|
|
Takashi Iwai |
c3ee3e |
+ else
|
|
Takashi Iwai |
c3ee3e |
+ dev_kfree_skb_any(skb);
|
|
Takashi Iwai |
c3ee3e |
|
|
Takashi Iwai |
c3ee3e |
return len;
|
|
Takashi Iwai |
c3ee3e |
}
|
|
Takashi Iwai |
c3ee3e |
--
|
|
Takashi Iwai |
c3ee3e |
2.16.4
|
|
Takashi Iwai |
c3ee3e |
|