Commit ee28b30c authored Dec 09, 2018 by Heiner Kallweit Committed by David S. Miller Dec 11, 2018

r8169: fix crash if CONFIG_DEBUG_SHIRQ is enabled



If CONFIG_DEBUG_SHIRQ is enabled __free_irq() intentionally fires
a spurious interrupt. This interrupt causes a crash because
tp->dev->phydev is NULL at that time.

Fixes: 38caff5a ("r8169: handle all interrupt events in the hard irq handler")
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent 2f1a9f66

drivers/net/ethernet/realtek/r8169.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -6469,7 +6469,7 @@ static irqreturn_t rtl8169_interrupt(int irq, void *dev_instance)
		goto out;
		}

		if (status & LinkChg)
		if (status & LinkChg && tp->dev->phydev)
		phy_mac_interrupt(tp->dev->phydev);

		if (unlikely(status & RxFIFOOver &&

Admin message