can: m_can: m_can_tx_work_queue(): fix tx_skb race condition
authorMarc Kleine-Budde <mkl@pengutronix.de>
Wed, 5 May 2021 11:32:27 +0000 (13:32 +0200)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Thu, 6 May 2021 07:24:07 +0000 (09:24 +0200)
commite04b2cfe61072c7966e1a5fb73dd1feb30c206ed
tree75e4c665edffabc8bd7631af5dfbf797f4d3a580
parent03c427147b2d3e503af258711af4fc792b89b0af
can: m_can: m_can_tx_work_queue(): fix tx_skb race condition

The m_can_start_xmit() function checks if the cdev->tx_skb is NULL and
returns with NETDEV_TX_BUSY in case tx_sbk is not NULL.

There is a race condition in the m_can_tx_work_queue(), where first
the skb is send to the driver and then the case tx_sbk is set to NULL.
A TX complete IRQ might come in between and wake the queue, which
results in tx_skb not being cleared yet.

Fixes: f524f829b75a ("can: m_can: Create a m_can platform framework")
Tested-by: Torin Cooper-Bennun <torin@maxiluxsystems.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/m_can/m_can.c