tree c384e7e37388ac2ac5e2c131365dc082cd604af7
parent 7b9e34e82120641953ece4cc6a8e2553575d7149
author Maxim Petrov <mmrmaximuzz@gmail.com> 1588573603 +0300
committer Greg Kroah-Hartman <gregkh@linuxfoundation.org> 1590593854 +0200

stmmac: fix pointer check after utilization in stmmac_interrupt

[ Upstream commit f42234ffd531ca6b13d9da02faa60b72eccf8334 ]

The paranoidal pointer check in IRQ handler looks very strange - it
really protects us only against bogus drivers which request IRQ line
with null pointer dev_id. However, the code fragment is incorrect
because the dev pointer is used before the actual check which leads
to undefined behavior. Remove the check to avoid confusing people
with incorrect code.

Signed-off-by: Maxim Petrov <mmrmaximuzz@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>